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 /hw/mem/memory-device.c | |
| parent | bc0e339b01b330c55532571ae6fce89b0f8a4a96 (diff) | |
| download | focaccia-qemu-f6166a4da59bdd57b3af8b1c742ce611d6c3d3aa.tar.gz focaccia-qemu-f6166a4da59bdd57b3af8b1c742ce611d6c3d3aa.zip | |
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 'hw/mem/memory-device.c')
0 files changed, 0 insertions, 0 deletions