commit acd5423aecf3f2f9694f64d2e3ca625462b19332
author Patrick Venture <venture@google.com> Tue Oct 16 16:58:59 2018 -0700
committer Patrick Venture <venture@google.com> Tue Oct 16 17:00:11 2018 -0700
tree 5c9445e4e71538d63fefd0be2107cdc29103feaa
parent 216108f1da47112268dc87f8984bb86db33c27d7

cleanup: use memcpy with fixed-size structures

Use std::memcpy with fixed-size structures.

Change-Id: I569be29fc7d1dc058b67936d030afbbbbad4bd0c
Signed-off-by: Patrick Venture <venture@google.com>

psu.cpp

diff --git a/psu.cpp b/psu.cpp
index 46e097a..1e46adc 100644
--- a/psu.cpp
+++ b/psu.cpp
@@ -47,14 +47,15 @@
 ipmi_ret_t PsuHardReset(const uint8_t* reqBuf, uint8_t* replyBuf,
                         size_t* dataLen)
 {
-    if ((*dataLen) < sizeof(struct PsuResetRequest))
+    struct PsuResetRequest request;
+
+    if ((*dataLen) < sizeof(request))
     {
         std::fprintf(stderr, "Invalid command length: %u\n",
                      static_cast<uint32_t>(*dataLen));
         return IPMI_CC_INVALID;
     }
 
-    struct PsuResetRequest request;
     std::memcpy(&request, &reqBuf[0], sizeof(struct PsuResetRequest));
 
     std::ofstream ofs;