commit e1ff81fa6ca45da17b0221fe1ba37ac30be66cc5
author Karthikeyan Pasupathi <pkarthikeyan1509@gmail.com> Mon Nov 21 17:54:46 2022 +0530
committer Patrick Williams <patrick@stwcx.xyz> Thu Dec 15 19:07:11 2022 +0000
tree a5ddecef8154dbe36ae9864b787f5bd22b46342e
parent e15e1ea963a7671d8591050cfe572e5db67b31fe

usb-dbg: Handled error for host selector position

Added error handling for host selector position
and enhancement.

Tested on Yosemitev2

Signed-off-by: Karthikeyan Pasupathi <pkarthikeyan1509@gmail.com>
Change-Id: Iac9414ccc282bd3e24b6434f0ae57dac586659a6

src/oemcommands.cpp

diff --git a/src/oemcommands.cpp b/src/oemcommands.cpp
index 2ee1e98..50e88b5 100644
--- a/src/oemcommands.cpp
+++ b/src/oemcommands.cpp
@@ -48,7 +48,7 @@
 
 using namespace phosphor::logging;
 
-size_t getSelectorPosition();
+void getSelectorPosition(size_t& position);
 static void registerOEMFunctions() __attribute__((constructor));
 sdbusplus::bus_t dbus(ipmid_get_sd_bus_connection()); // from ipmid/api.h
 static constexpr size_t maxFRUStringLength = 0x3F;
@@ -359,7 +359,7 @@
     bool platform = isMultiHostPlatform();
     if (platform == true)
     {
-        pos = getSelectorPosition();
+        getSelectorPosition(pos);
     }
 
     sd_bus* bus = NULL;