commit b84ccc614d2032b5ddee9ab145d82d2eb70723e8
author Dave Barach <dave@barachs.net> Thu Jun 16 20:14:43 2016 -0400
committer Damjan Marion <damarion@cisco.com> Fri Jun 17 07:09:44 2016 +0000
tree 37d4bc534c67027aa4e65a02d1444130282307ab
parent 45050f8781a4ceb8b99068c75604b81a32b9cf84

vpp-83: fix double-unlock

Change-Id: I0b75b4f9bef62aba69e9cc163924f6b985a35455
Signed-off-by: Dave Barach <dave@barachs.net>

vlib-api/vlibmemory/memory_shared.c

diff --git a/vlib-api/vlibmemory/memory_shared.c b/vlib-api/vlibmemory/memory_shared.c
index 1684acf..e4f5dd0 100644
--- a/vlib-api/vlibmemory/memory_shared.c
+++ b/vlib-api/vlibmemory/memory_shared.c
@@ -288,8 +288,9 @@
             pthread_mutex_lock (&root_rp->mutex);
             svm_client_scan_this_region_nolock (root_rp);
             pthread_mutex_unlock (&root_rp->mutex);
-        } 
-        pthread_mutex_unlock (&vlib_rp->mutex);
+        } else {
+            pthread_mutex_unlock (&vlib_rp->mutex);
+        }
         am->vlib_rp = vlib_rp;
         vec_add1(am->mapped_shmem_regions, vlib_rp);
         return 0;