commit 64a350d3214e8bb9b43c1ff4d49b29ad38ad0d04
author Vernon Mauery <vernon.mauery@linux.intel.com> Wed Mar 06 09:24:55 2019 -0800
committer Vernon Mauery <vernon.mauery@linux.intel.com> Thu Mar 07 14:40:50 2019 +0000
tree 0a369925f4178926beffc22bace5d8ca796ec98f
parent 30206dbaa6eb0cbddd7e215339f8a112ca011d6a

fix whitelist RestrictionMode lookup

The whitelist RestrictionMode lookup was missing the interface name on
the properties get method call. This is a fixup for that.

Tested-by: watch ipmid startup and see that it does not complain about
           fetching the whitelist setting.

Change-Id: I030c0c66771afcb987e0679595a2d662f7c3ea9e
Signed-off-by: Vernon Mauery <vernon.mauery@linux.intel.com>

whitelist-filter.cpp

diff --git a/whitelist-filter.cpp b/whitelist-filter.cpp
index 6fe0f3e..3b45db8 100644
--- a/whitelist-filter.cpp
+++ b/whitelist-filter.cpp
@@ -6,6 +6,7 @@
 #include <phosphor-logging/elog-errors.hpp>
 #include <phosphor-logging/log.hpp>
 #include <settings.hpp>
+#include <utils.hpp>
 #include <xyz/openbmc_project/Control/Security/RestrictionMode/server.hpp>
 
 using namespace phosphor::logging;
@@ -81,7 +82,7 @@
         return;
     }
     bus->async_method_call(
-        [this](boost::system::error_code ec, std::string mode) {
+        [this](boost::system::error_code ec, ipmi::Value v) {
             if (ec)
             {
                 log<level::ERR>("Error in RestrictionMode Get");
@@ -89,6 +90,7 @@
                 restrictedMode = true;
                 return;
             }
+            auto mode = std::get<std::string>(v);
             auto restrictionMode =
                 RestrictionMode::convertModesFromString(mode);
             restrictedMode =
@@ -96,8 +98,9 @@
             log<level::INFO>((restrictedMode ? "Set restrictedMode = true"
                                              : "Set restrictedMode = false"));
         },
-        restrictionModeService.c_str(), restrictionModeSetting.c_str(),
-        "org.freedesktop.DBus.Properties", "Get", "RestrictionMode");
+        restrictionModeService, restrictionModeSetting,
+        "org.freedesktop.DBus.Properties", "Get", restrictionModeIntf,
+        "RestrictionMode");
 }
 
 void WhitelistFilter::handleRestrictedModeChange(sdbusplus::message::message& m)