diff options
| author | Markus Armbruster <armbru@redhat.com> | 2019-12-04 10:36:15 +0100 |
|---|---|---|
| committer | Markus Armbruster <armbru@redhat.com> | 2019-12-18 08:36:15 +0100 |
| commit | f6166a4da59bdd57b3af8b1c742ce611d6c3d3aa (patch) | |
| tree | b3b099cd5fe728caf2223fd965873a51ffa22ade /qga | |
| parent | bc0e339b01b330c55532571ae6fce89b0f8a4a96 (diff) | |
hw/ipmi: Fix realize() error API violations
isa_ipmi_bt_realize(), ipmi_isa_realize(), pci_ipmi_bt_realize(), and
pci_ipmi_kcs_realize() dereference @errp when IPMIInterfaceClass
method init() fails. That's wrong; see the big comment in error.h.
Introduced in commit 0719029c47 "ipmi: Add an ISA KCS low-level
interface", then imitated in commit a9b74079cb "ipmi: Add a BT
low-level interface" and commit 12f983c6aa "ipmi: Add PCI IPMI
interfaces".
No caller actually passes null.
Fix anyway: splice in a local Error *err, and error_propagate().
Cc: Corey Minyard <cminyard@mvista.com>
Signed-off-by: Markus Armbruster <armbru@redhat.com>
Message-Id: <20191204093625.14836-9-armbru@redhat.com>
Diffstat (limited to 'qga')
0 files changed, 0 insertions, 0 deletions