[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] weird qdev error
From: |
Michael S. Tsirkin |
Subject: |
Re: [Qemu-devel] weird qdev error |
Date: |
Mon, 13 Feb 2012 06:58:08 +0200 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
On Mon, Feb 13, 2012 at 03:18:13AM +0200, Michael S. Tsirkin wrote:
> > I also see this:
> >
> > device_add virtio-net-pci,netdev=foo,mac=52:54:00:12:34:56,id=bla
> > device_del bla
> > *** glibc detected *** /home/mst/qemu-test/bin/qemu-system-x86_64:
> > corrupted double-linked list: 0x00007fae434565a0 ***
> >
> > Am I alone?
Doesn't solve this issue, but shouldn't we use _SAFE
in object_property_del_child? Like this:
--->
qemu: use safe list macro
As we might remove an element from list, use the safe macro
to walk it.
Signed-off-by: Michael S. Tsirkin <address@hidden>
---
diff --git a/qom/object.c b/qom/object.c
index 5e5b261..8b64fb6 100644
--- a/qom/object.c
+++ b/qom/object.c
@@ -299,9 +299,9 @@ static void object_property_del_all(Object *obj)
static void object_property_del_child(Object *obj, Object *child, Error **errp)
{
- ObjectProperty *prop;
+ ObjectProperty *prop, *next;
- QTAILQ_FOREACH(prop, &obj->properties, node) {
+ QTAILQ_FOREACH_SAFE(prop, &obj->properties, node, next) {
if (!strstart(prop->type, "child<", NULL)) {
continue;
}
- [Qemu-devel] weird qdev error, Michael S. Tsirkin, 2012/02/12
- Re: [Qemu-devel] weird qdev error, Michael S. Tsirkin, 2012/02/12
- Re: [Qemu-devel] weird qdev error, Anthony Liguori, 2012/02/12
- Re: [Qemu-devel] weird qdev error, Michael S. Tsirkin, 2012/02/12
- Re: [Qemu-devel] weird qdev error, Anthony Liguori, 2012/02/12
- Re: [Qemu-devel] weird qdev error, Michael S. Tsirkin, 2012/02/12
- Re: [Qemu-devel] weird qdev error, Anthony Liguori, 2012/02/12
- Re: [Qemu-devel] weird qdev error, Michael S. Tsirkin, 2012/02/12
- Re: [Qemu-devel] weird qdev error, Michael S. Tsirkin, 2012/02/12
- Re: [Qemu-devel] weird qdev error,
Michael S. Tsirkin <=
- Re: [Qemu-devel] weird qdev error, Paolo Bonzini, 2012/02/13