From 40e76aa4f07432338a0c8efbb16a10c7cc763d12 Mon Sep 17 00:00:00 2001
From: Kefu Chai <kchai@redhat.com>
Date: Fri, 27 Jul 2018 12:13:42 +0800
Subject: [PATCH] common/config: fix the lock in ConfigProxy::diff()

it's a regression introduced by e406d8eb9e1deb801ecb346169eaaf96adbb4b53

Signed-off-by: Kefu Chai <kchai@redhat.com>
---
 src/common/config_proxy.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/common/config_proxy.h b/src/common/config_proxy.h
index 2bcc7a44e6c9..d6f75b10b87f 100644
--- a/src/common/config_proxy.h
+++ b/src/common/config_proxy.h
@@ -71,8 +71,8 @@ public:
     return config.find_option(name);
   }
   void diff(Formatter *f, const std::string& name=string{}) const {
-    return config.diff(values, f, name);
     Mutex::Locker l{lock};
+    return config.diff(values, f, name);
   }
   void get_my_sections(std::vector <std::string> &sections) const {
     Mutex::Locker l{lock};
-- 
2.47.3