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 /dcmihandler.cpp | |
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 'dcmihandler.cpp')
0 files changed, 0 insertions, 0 deletions