commit 6d76560589faef9e8d5807827779b395c937a3c9
author Patrick Venture <venture@google.com> Wed Sep 25 17:11:07 2019 -0700
committer Patrick Venture <venture@google.com> Thu Sep 26 16:37:35 2019 +0000
tree 80790c058299378932b51c3af3199692fb13970e
parent 378903928085318c204952400754d4a82b47f1e5

bugfix: smbiosmdrv2handler: sizeof(pointer)

[src/smbiosmdrv2handler.cpp:570]:
[src/smbiosmdrv2handler.cpp:1002]:
[src/smbiosmdrv2handler.cpp:1112]:
(warning) Using 'sizeof' on array given as function argument returns
size of a pointer.

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

src/smbiosmdrv2handler.cpp

diff --git a/src/smbiosmdrv2handler.cpp b/src/smbiosmdrv2handler.cpp
index 52d23ad..1f5201d 100644
--- a/src/smbiosmdrv2handler.cpp
+++ b/src/smbiosmdrv2handler.cpp
@@ -567,7 +567,7 @@
         return ipmi::responseParmOutOfRange();
     }
 
-    int idIndex = mdrv2->findDataId(dataInfo.data(), sizeof(dataInfo), service);
+    int idIndex = mdrv2->findDataId(dataInfo.data(), dataInfo.size(), service);
 
     if ((idIndex < 0) || (idIndex >= maxDirEntries))
     {
@@ -999,7 +999,7 @@
     std::shared_ptr<sdbusplus::asio::connection> bus = getSdBus();
     std::string service = ipmi::getService(*bus, mdrv2Interface, mdrv2Path);
 
-    int idIndex = mdrv2->findDataId(dataInfo.data(), sizeof(dataInfo), service);
+    int idIndex = mdrv2->findDataId(dataInfo.data(), dataInfo.size(), service);
 
     if ((idIndex < 0) || (idIndex >= maxDirEntries))
     {
@@ -1109,7 +1109,7 @@
         return ipmi::responseParmOutOfRange();
     }
 
-    int idIndex = mdrv2->findDataId(dataInfo.data(), sizeof(dataInfo), service);
+    int idIndex = mdrv2->findDataId(dataInfo.data(), dataInfo.size(), service);
 
     if ((idIndex < 0) || (idIndex >= maxDirEntries))
     {