On Fri, Jul 13, 2018 at 10:27:37AM +0200, Thomas Huth wrote: > Running QEMU with valgrind indicates a problem here: > > echo "{'execute':'qmp_capabilities'} {'execute':'device-list-properties'," \ > "'arguments':{'typename':'a9mpcore_priv'}}" \ > "{'execute': 'human-monitor-command', " \ > "'arguments': {'command-line': 'info qtree'}}" | \ > valgrind -q aarch64-softmmu/qemu-system-aarch64 -M none,accel=qtest -qmp > stdio > [...] > ==30996== Invalid read of size 8 > ==30996== at 0x6185DA: qdev_print (qdev-monitor.c:686) > ==30996== by 0x6185DA: qbus_print (qdev-monitor.c:719) > ==30996== by 0x452B38: handle_hmp_command (monitor.c:3446) > [...] > > Use the new sysbus_init_child_obj() function to make sure that the objects > are cleaned up correctly when the parent gets destroyed. > > Signed-off-by: Thomas Huth <th...@redhat.com>
Reviewed-by: Eduardo Habkost <ehabk...@redhat.com> -- Eduardo