diff options
| author | Andy Shevchenko <andriy.shevchenko@linux.intel.com> | 2019-01-24 21:22:20 +0100 |
|---|---|---|
| committer | Corey Minyard <cminyard@mvista.com> | 2019-02-10 02:48:43 +0100 |
| commit | f32043901a389ee8b5417c0cec547a84a2f9b38a (patch) | |
| tree | 8207111f3a8cd7259a6123263baabb1827e07db1 /drivers/char/ipmi | |
| parent | ipmi: Use defined constant for UUID representation (diff) | |
| download | linux-f32043901a389ee8b5417c0cec547a84a2f9b38a.tar.xz linux-f32043901a389ee8b5417c0cec547a84a2f9b38a.zip | |
ipmi: Use dedicated API for copying a UUID
Use guid_copy() instead of memcpy() to hide guid_t implementation details and
to show we expect guid_t in a raw buffer.
Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Signed-off-by: Corey Minyard <cminyard@mvista.com>
Diffstat (limited to 'drivers/char/ipmi')
| -rw-r--r-- | drivers/char/ipmi/ipmi_msghandler.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/char/ipmi/ipmi_msghandler.c b/drivers/char/ipmi/ipmi_msghandler.c index 79911914c1cb..e8ba67834746 100644 --- a/drivers/char/ipmi/ipmi_msghandler.c +++ b/drivers/char/ipmi/ipmi_msghandler.c @@ -3156,7 +3156,7 @@ static void guid_handler(struct ipmi_smi *intf, struct ipmi_recv_msg *msg) goto out; } - memcpy(bmc->fetch_guid.b, msg->msg.data + 1, 16); + guid_copy(&bmc->fetch_guid, (guid_t *)(msg->msg.data + 1)); /* * Make sure the guid data is available before setting * dyn_guid_set. |