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.shevche...@linux.intel.com>
---
 drivers/char/ipmi/ipmi_msghandler.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/char/ipmi/ipmi_msghandler.c 
b/drivers/char/ipmi/ipmi_msghandler.c
index 6a27fccd9f36..2467497b525b 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.
-- 
2.20.1



_______________________________________________
Openipmi-developer mailing list
Openipmi-developer@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/openipmi-developer

Reply via email to