commit 1dd18d2466e7952a0d27b3e5c9fa18e6e91336e6
author Prithvi Pai <ppai@nvidia.com> Fri Sep 19 14:27:03 2025 +0530
committer Prithvi Pai <ppai@nvidia.com> Mon Sep 22 14:09:45 2025 +0000
tree 792e984d0b49597ee9303773a4cb9dd11dcca043
parent 400695c4c1d1c56b58bc22931c9b3bc60cfe0972

Add fix for GetChannelPaylodSupport

Currently `getChannelName` throws when the channel number
is invalid.
Because of which the error reported is Unspecified.

Example:
root@gb200nvl-obmc:~# ipmitool raw 0x06 0x4E 0x4
[1] Unable to send RAW command (channel=0x0 netfn=0x6 lun=0x0 cmd=0x4e rsp=0xff): Unspecified error

Added a patch to handle this gracefully

Tested:
Tested on gb200nvl-obmc platform
root@gb200nvl-obmc:~# ipmitool raw 0x06 0x4E 0x4
[2] Unable to send RAW command (channel=0x0 netfn=0x6 lun=0x0 cmd=0x4e rsp=0xcc): Invalid data field in request

Change-Id: Ibc9981d5e7143311f28bb9191fed4d68cd81356b
Signed-off-by: Prithvi Pai <ppai@nvidia.com>

dcmihandler.cpp

diff --git a/dcmihandler.cpp b/dcmihandler.cpp
index e6cb102..c02c407 100644
--- a/dcmihandler.cpp
+++ b/dcmihandler.cpp
@@ -517,6 +517,12 @@
     ipmi::Context::ptr& ctx)
 {
     auto ethdevice = ipmi::getChannelName(ethernetDefaultChannelNum);
+    if (ethdevice.empty())
+    {
+        lg2::error("Channel name does not exist for channel {CHANNEL}",
+                   "CHANNEL", ethernetDefaultChannelNum);
+        return std::nullopt;
+    }
     ipmi::DbusObjectInfo ethernetObj{};
     boost::system::error_code ec = ipmi::getDbusObject(
         ctx, ethernetIntf, networkRoot, ethdevice, ethernetObj);