diff options
| author | Markus Armbruster <armbru@redhat.com> | 2018-08-06 08:53:31 +0200 |
|---|---|---|
| committer | Markus Armbruster <armbru@redhat.com> | 2018-08-16 08:42:06 +0200 |
| commit | 2d36e843042d2ef47f3bfc47a1a83401fdb07b84 (patch) | |
| tree | cf0c271134caf76933cb959864404f46e6695341 /hw/misc/mst_fpga.c | |
| parent | eac78bd430f357b9d935afaa09355c6fc6a13e3d (diff) | |
| download | focaccia-qemu-2d36e843042d2ef47f3bfc47a1a83401fdb07b84.tar.gz focaccia-qemu-2d36e843042d2ef47f3bfc47a1a83401fdb07b84.zip | |
qobject: qobject_from_jsonv() is dangerous, hide it away
qobject_from_jsonv() takes ownership of %p arguments. On failure, we can't generally know whether we failed before or after %p, so ownership becomes indeterminate. To avoid leaks, callers passing %p must terminate on error, e.g. by passing &error_abort. Trap for the unwary; document and give the function internal linkage. Signed-off-by: Markus Armbruster <armbru@redhat.com> Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org> Reviewed-by: Eric Blake <eblake@redhat.com> Message-Id: <20180806065344.7103-11-armbru@redhat.com>
Diffstat (limited to 'hw/misc/mst_fpga.c')
0 files changed, 0 insertions, 0 deletions