Am 27.06.2012 14:41, schrieb Anthony Liguori: > From Markus: > > Before: > > $ qemu-system-x86_64 -display none -drive if=ide > qemu-system-x86_64: Device needs media, but drive is empty > qemu-system-x86_64: Initialization of device ide-hd failed > [Exit 1 ] > > After: > > $ qemu-system-x86_64 -display none -drive if=ide > qemu-system-x86_64: Device needs media, but drive is empty > Segmentation fault (core dumped) > [Exit 139 (SIGSEGV)] > > This error always existed as qdev_init() frees the object. But QOM > goes a bit further and purposefully sets the class pointer to NULL to > help find use-after-free. It worked :-) > > Cc: Andreas Faerber <afaer...@suse.de> > Reported-by: Markus Armbruster <arm...@redhat.com> > Signed-off-by: Anthony Liguori <aligu...@us.ibm.com>
Reviewed-by: Andreas Färber <afaer...@suse.de> This together with the semantics discussions we're having makes me think we should attack "QOM'ifying" qdev sooner than later. I.e., reviewing what naming, chaining, etc. we can already change to align the TYPE_DEVICE-derived types with the generic QOM infrastructure. Andreas -- SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg
