Re: [libvirt] [PATCH 1/2] conf: Fix race between looking up a domain object and freeing it

2013-04-10 Thread Peter Krempa
On 04/09/13 16:52, Eric Blake wrote: On 04/09/2013 07:02 AM, Peter Krempa wrote: This patch fixes crash of the daemon that happens due to the following race condition: ... --- src/conf/domain_conf.c | 4 1 file changed, 4 insertions(+) Commit message needs help, but the code is co

Re: [libvirt] [PATCH 1/2] conf: Fix race between looking up a domain object and freeing it

2013-04-09 Thread Eric Blake
On 04/09/2013 07:02 AM, Peter Krempa wrote: > This patch fixes crash of the daemon that happens due to the following race > condition: > > Let's have two threads in the libvirtd daemon's qemu driver: > A - thread executing a API call to get information about a domain > B - thread executing undefin

Re: [libvirt] [PATCH 1/2] conf: Fix race between looking up a domain object and freeing it

2013-04-09 Thread Viktor Mihajlovski
On 04/09/2013 03:02 PM, Peter Krempa wrote: 1 file changed, 4 insertions(+) diff --git a/src/conf/domain_conf.c b/src/conf/domain_conf.c index 03e5740..cafef0c 100644 --- a/src/conf/domain_conf.c +++ b/src/conf/domain_conf.c @@ -2238,10 +2238,14 @@ void virDomainObjListRemove(virDomainObjListP

[libvirt] [PATCH 1/2] conf: Fix race between looking up a domain object and freeing it

2013-04-09 Thread Peter Krempa
This patch fixes crash of the daemon that happens due to the following race condition: Let's have two threads in the libvirtd daemon's qemu driver: A - thread executing a API call to get information about a domain B - thread executing undefine on the same domain Assume following serialization of