commit 0922bde4e5a371b6493f178071f252c195f71e81
author Jayanth Othayoth <ojayanth@in.ibm.com> Mon Apr 02 07:59:34 2018 -0500
committer Vernon Mauery <vernon.mauery@linux.intel.com> Wed Apr 11 20:59:14 2018 +0000
tree 99a4af15a22596b51a07a6e144c1da000979dccd
parent 3d23048896b7171b940c56fe4e403b1d23103eef

sensorhandler: Enable mutability based set function

Resolves openbmc/openbmc#2980

Change-Id: I67afd4e84ec96e5cfc2dd315604d70f41c67a438
Signed-off-by: Jayanth Othayoth <ojayanth@in.ibm.com>

sensorhandler.cpp

diff --git a/sensorhandler.cpp b/sensorhandler.cpp
index 3a47e20..1a66a1e 100644
--- a/sensorhandler.cpp
+++ b/sensorhandler.cpp
@@ -379,6 +379,13 @@
 
     try
     {
+        if (ipmi::sensor::Mutability::Write !=
+              (iter->second.mutability & ipmi::sensor::Mutability::Write))
+        {
+            log<level::ERR>("Sensor Set operation is not allowed",
+                            entry("SENSOR_NUM=%d", cmdData.number));
+            return IPMI_CC_ILLEGAL_COMMAND;
+        }
         return iter->second.updateFunc(cmdData, iter->second);
     }
     catch (InternalFailure& e)