diff options
| author | Vernon Mauery <vernon.mauery@linux.intel.com> | 2019-05-08 15:19:05 -0700 |
|---|---|---|
| committer | Vernon Mauery <vernon.mauery@linux.intel.com> | 2019-05-14 15:42:47 +0000 |
| commit | 23b7021c00b0a52cd7837c9d51cfa7fa52dc9776 (patch) | |
| tree | b538d48fc806660d5ae4997625fd3b2ffc1a302a /user_channel/channel_mgmt.hpp | |
| parent | 687df40108b0103f7539fafebcce63c1f8405335 (diff) | |
| download | phosphor-host-ipmid-23b7021c00b0a52cd7837c9d51cfa7fa52dc9776.tar.gz phosphor-host-ipmid-23b7021c00b0a52cd7837c9d51cfa7fa52dc9776.zip | |
Legacy D-Bus interface must set yield_context
The legacy D-Bus interface was not setting the yield_context pointer
in the ipmi::Context struct. This means that a command coming in from
a legacy D-Bus interface that called a new-provider that uses the
yield_method_call function would segfault on a NULL pointer. This
change makes the legacy interface behave the same as the new
D-Bus interface by creating a new boost::coroutine for each command.
Tested-by: running a legacy ipmitool to ensure that the rewritten set
chassis power restore policy command worked without a
segfault.
Change-Id: Ie45fff1fc222b9b949a9d3976bf9d1b2c29bd8f5
Signed-off-by: Vernon Mauery <vernon.mauery@linux.intel.com>
Diffstat (limited to 'user_channel/channel_mgmt.hpp')
0 files changed, 0 insertions, 0 deletions
