summaryrefslogtreecommitdiff
path: root/vswitchd/bridge.c
diff options
context:
space:
mode:
authorBen Pfaff <blp@nicira.com>2013-05-16 22:37:53 -0700
committerBen Pfaff <blp@nicira.com>2013-05-16 22:49:21 -0700
commit2291eb080c1afe13a5d3cb3048f0d160386b714b (patch)
treea9788667333fd9bdbf573b1ab997d9c751eccb12 /vswitchd/bridge.c
parent63a10d0e099fe8eb6052fadf6b80a0ab208a6381 (diff)
downloadopenvswitch-2291eb080c1afe13a5d3cb3048f0d160386b714b.tar.gz
bridge: Fix use of wild pointer in iface_do_create().
'netdev' was not initialized at this point in the function, so the cleanup at the error label referenced a wild pointer. Introduced in commit 94a538422d4b (netdev: Prevent using reserved names). CC: Alex Wang <alexw@nicira.com> Signed-off-by: Ben Pfaff <blp@nicira.com>
Diffstat (limited to 'vswitchd/bridge.c')
-rw-r--r--vswitchd/bridge.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/vswitchd/bridge.c b/vswitchd/bridge.c
index 28e306ea4..e10036c9a 100644
--- a/vswitchd/bridge.c
+++ b/vswitchd/bridge.c
@@ -1415,7 +1415,7 @@ iface_do_create(const struct bridge *br,
{
const struct ovsrec_interface *iface_cfg = if_cfg->cfg;
const struct ovsrec_port *port_cfg = if_cfg->parent;
- struct netdev *netdev;
+ struct netdev *netdev = NULL;
int error;
if (netdev_is_reserved_name(iface_cfg->name)) {