From 4cecbfa01ea105d1b8dfb5db5ee2d60fbda0be80 Mon Sep 17 00:00:00 2001
From: Gunnar Beutner <gunnar.beutner@netways.de>
Date: Sat, 2 Mar 2013 14:11:12 +0100
Subject: [PATCH] Fixed deadlock in ConfigItem::Commit().

---
 lib/config/configitem.cpp | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/lib/config/configitem.cpp b/lib/config/configitem.cpp
index 1d0893f38..998bfff7f 100644
--- a/lib/config/configitem.cpp
+++ b/lib/config/configitem.cpp
@@ -217,11 +217,13 @@ DynamicObject::Ptr ConfigItem::Commit(void)
 		m_ChildObjects = children;
 	}
 
+	ConfigItem::Ptr self = GetSelf();
+
 	{
 		recursive_mutex::scoped_lock lock(m_Mutex);
 
 		/* Register this item. */
-		m_Items[ikey] = GetSelf();
+		m_Items[ikey] = self;
 	}
 
 	DynamicObject::Ptr dobj = GetDynamicObject();
@@ -232,8 +234,7 @@ DynamicObject::Ptr ConfigItem::Commit(void)
 	/* Register this item with its parents. */
 	BOOST_FOREACH(const String& parentName, GetParents()) {
 		ConfigItem::Ptr parent = GetObject(GetType(), parentName);
-		ObjectLock olock(parent);
-		parent->RegisterChild(GetSelf());
+		parent->RegisterChild(self);
 	}
 
 	/* Create a fake update in the format that