fix:Channel access mode set to disabled by default
The recent commit ac05aa1cdeb0d5be0baae098c4187166ea026976 is reloading
the channel config data alone when network interfaces added signals received.
This results in clearing ChannelProperties completely and missed to fill
ChannelAccess data back. Which broke IPMI over RMCP+.
Update the same to call initChannelPersistData() instead so that complete
channel config data can be reloaded.
Tested:
Verified Access Mode value is populated correctly.
IPMI commands over RMCP+ worked as expected.
Signed-off-by: Arun P. Mohanan <arun.p.m@linux.intel.com>
Change-Id: I45105f7a14ab9c32589f1f9f972ab2572d274877
diff --git a/user_channel/channel_mgmt.cpp b/user_channel/channel_mgmt.cpp
index 5797a19..aa746fd 100644
--- a/user_channel/channel_mgmt.cpp
+++ b/user_channel/channel_mgmt.cpp
@@ -364,7 +364,9 @@
sdbusplus::bus::match::rules::member(interfaceAddedSignal) +
sdbusplus::bus::match::rules::argNpath(
0, std::string(networkIntfObjectBasePath) + "/"),
- [&](sdbusplus::message::message& msg) { loadChannelConfig(); });
+ [&](sdbusplus::message::message& msg) {
+ initChannelPersistData();
+ });
chInterfaceRemovedSignal = std::make_unique<sdbusplus::bus::match_t>(
bus,
@@ -372,7 +374,9 @@
sdbusplus::bus::match::rules::member(interfaceRemovedSignal) +
sdbusplus::bus::match::rules::argNpath(
0, std::string(networkIntfObjectBasePath) + "/"),
- [&](sdbusplus::message::message& msg) { loadChannelConfig(); });
+ [&](sdbusplus::message::message& msg) {
+ initChannelPersistData();
+ });
}
}