commit 14c154678b0a22825bfd5701db6031c3feab7ce7
author Tom Joseph <tomjoseph@in.ibm.com> Thu Sep 07 02:16:51 2017 +0530
committer Tom Joseph <tomjoseph@in.ibm.com> Thu Sep 14 19:24:34 2017 +0530
tree 5e44e8e8491f478f38244b20272baafc07830308
parent 40c35b1c830c8c903c0120b074a3b7bc631a9861

sensor: Use getFunc to handle the get sensor reading

Change-Id: If0f3185e7ce1d300be20b58ee5004a58e35c222d
Signed-off-by: Tom Joseph <tomjoseph@in.ibm.com>

sensorhandler.cpp

diff --git a/sensorhandler.cpp b/sensorhandler.cpp
index 1aa5181..ca8ef53 100644
--- a/sensorhandler.cpp
+++ b/sensorhandler.cpp
@@ -663,9 +663,28 @@
             break;
         }
         default:
-            *data_len=0;
-            rc = IPMI_CC_SENSOR_INVALID;
-            break;
+        {
+            const auto iter = sensors.find(reqptr->sennum);
+            if (iter == sensors.end())
+            {
+                return IPMI_CC_SENSOR_INVALID;
+            }
+
+            try
+            {
+                auto getResponse =  iter->second.getFunc(iter->second);
+                *data_len = getResponse.size();
+                memcpy(resp, getResponse.data(), *data_len);
+                return IPMI_CC_OK;
+            }
+            catch (InternalFailure& e)
+            {
+                 log<level::ERR>("Get sensor failed",
+                                 entry("SENSOR_NUM=%d", reqptr->sennum));
+                 commit<InternalFailure>();
+                 return IPMI_CC_SENSOR_INVALID;
+            }
+        }
     }
 
     reply = sd_bus_message_unref(reply);