ovs-vsctl: Avoid redundant initializations in "add-br" command.
authorBen Pfaff <blp@nicira.com>
Wed, 10 Feb 2010 19:21:56 +0000 (11:21 -0800)
committerBen Pfaff <blp@nicira.com>
Thu, 11 Feb 2010 18:35:28 +0000 (10:35 -0800)
Found by Clang (http://clang-analyzer.llvm.org).

utilities/ovs-vsctl.c

index 9baca4286205116a0247d7f898fa849c34c5cb1e..498a0b0b3e21522f61aadd7bdf2b20fd3ec6036f 100644 (file)
@@ -832,10 +832,9 @@ static void
 cmd_add_br(struct vsctl_context *ctx)
 {
     bool may_exist = shash_find(&ctx->options, "--may-exist") != 0;
-    const char *br_name = ctx->argv[1];
-    const char *parent_name = ctx->argc > 2 ? ctx->argv[2] : NULL;
-    int vlan = ctx->argc > 3 ? atoi(ctx->argv[3]) : 0;
+    const char *br_name, *parent_name;
     struct vsctl_info info;
+    int vlan;
 
     br_name = ctx->argv[1];
     if (ctx->argc == 2) {