Change signature of getService

getService() has [bus, interface, path] as parameter and
the caller invoked it with [bus, path, interface] and that
resulted in a failure. Although the fix can be put into the
caller to pass the right arguments, better fix would be to
change the signature of getService() to receive path and then
the interface as that is what has been followed all over and
the deviation resulted in this bug.

Change-Id: I3efe7f4f0a2d0bceb7e6e801cd68de6b2624fbe0
Signed-off-by: Vishwanatha Subbanna <vishwa@linux.vnet.ibm.com>

2 files changed, 5 insertions, 5 deletions

diff --git a/utils.cpp b/utils.cpp
index 1dfdcef..d9a98ee 100644
--- a/utils.cpp
+++ b/utils.cpp
@@ -13,8 +13,8 @@ using InternalFailure = sdbusplus::xyz::openbmc_project::Common::
                             Error::InternalFailure;
 
 std::string getService(sdbusplus::bus::bus& bus,
-                       const std::string& intf,
-                       const std::string& path)
+                       const std::string& path,
+                       const std::string& intf)
 {
     auto mapperCall = bus.new_method_call("xyz.openbmc_project.ObjectMapper",
                                           "/xyz/openbmc_project/object_mapper",
diff --git a/utils.hpp b/utils.hpp
index 51f9832..33088df 100644
--- a/utils.hpp
+++ b/utils.hpp
@@ -10,14 +10,14 @@ namespace occ
  * @brief Gets the D-Bus Service name for the input D-Bus path
  *
  * @param[in] bus  -  Bus handler
- * @param[in] intf -  Interface
  * @param[in] path -  Object Path
+ * @param[in] intf -  Interface
  *
  * @return            Service name
  * @error             InternalFailure exception thrown
  */
 std::string getService(sdbusplus::bus::bus& bus,
-                       const std::string& intf,
-                       const std::string& path);
+                       const std::string& path,
+                       const std::string& intf);
 } // namespace occ
 } // namespace open_power