From: Ben Pfaff <blp@nicira.com>
Date: Mon, 12 Aug 2013 19:51:47 +0000 (-0700)
Subject: netdev-dummy: Fix synchronization error in netdev_dummy_get_config().
X-Git-Url: http://git.onelab.eu/?a=commitdiff_plain;h=5a9bf5148ef91147ff1f24f31ab790928ee838da;p=sliver-openvswitch.git

netdev-dummy: Fix synchronization error in netdev_dummy_get_config().

Found by Clang.

Signed-off-by: Ben Pfaff <blp@nicira.com>
Acked-by: Andy Zhou <azhou@nicira.com>
---

diff --git a/lib/netdev-dummy.c b/lib/netdev-dummy.c
index 5c312109b..32b0943f9 100644
--- a/lib/netdev-dummy.c
+++ b/lib/netdev-dummy.c
@@ -327,12 +327,15 @@ netdev_dummy_get_config(const struct netdev *netdev_, struct smap *args)
 {
     struct netdev_dummy *netdev = netdev_dummy_cast(netdev_);
 
+    ovs_mutex_lock(&netdev->mutex);
     if (netdev->ifindex >= 0) {
         smap_add_format(args, "ifindex", "%d", netdev->ifindex);
     }
     if (netdev->pstream) {
         smap_add(args, "pstream", pstream_get_name(netdev->pstream));
     }
+    ovs_mutex_unlock(&netdev->mutex);
+
     return 0;
 }