From 43b897c17a5768efa717d18f522b96c090137268 Mon Sep 17 00:00:00 2001
From: "Daniel P. Berrange" <berrange@redhat.com>
Date: Mon, 15 Dec 2008 11:00:11 +0000
Subject: [PATCH] Fix deadlock when reloading configs

---
 ChangeLog         | 5 +++++
 src/domain_conf.c | 5 ++++-
 2 files changed, 9 insertions(+), 1 deletion(-)

diff --git a/ChangeLog b/ChangeLog
index 5a5f2c10d7..7371c8d702 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,8 @@
+Mon Dec 15 10:59:19 GMT 2008 Daniel P. Berrange <berrange@redhat.com>
+
+	* src/domain_conf.c: Unlock domain object after fetching
+	it to avoid deadlock when re-loading configs
+
 Fri Dec 12 17:41:19 +0100 2008 Jim Meyering <meyering@redhat.com>
 
 	remove redundant optname arguments
diff --git a/src/domain_conf.c b/src/domain_conf.c
index 485ffb2436..75f32e5b11 100644
--- a/src/domain_conf.c
+++ b/src/domain_conf.c
@@ -3330,8 +3330,11 @@ virDomainObjPtr virDomainLoadConfig(virConnectPtr conn,
                                       VIR_DOMAIN_XML_INACTIVE)))
         goto error;
 
-    if (virDomainFindByName(doms, def->name))
+    if ((dom = virDomainFindByName(doms, def->name))) {
+        virDomainObjUnlock(dom);
+        dom = NULL;
         newVM = 0;
+    }
 
     if (!(dom = virDomainAssignDef(conn, doms, def)))
         goto error;