Fix reading wrong value when bad file provided.
authorJustin Pettit <jpettit@nicira.com>
Wed, 1 Apr 2009 23:44:09 +0000 (16:44 -0700)
committerJustin Pettit <jpettit@nicira.com>
Wed, 1 Apr 2009 23:44:09 +0000 (16:44 -0700)
When a non-existent config file is specified, we print an error message.
Unfortunately, this was attempting to read the filename from the wrong
variable, so a random pointer was being treated as a string.

vswitchd/brcompatd.c
vswitchd/vswitchd.c

index c74e4f105190cf65f7b895c838e904613dc2fcc0..6e8b18c1b2c5b20ec97de036eb4cc6eb7c367bab 100644 (file)
@@ -675,7 +675,7 @@ parse_options(int argc, char *argv[])
     error = cfg_set_file(config_file);
     if (error) {
         ofp_fatal(error, "failed to add configuration file \"%s\"", 
-                optarg);
+                config_file);
     }
 }
 
index ce0d68e0dd15101f2bf736d080cf006e63123ee5..669a8c42cd11732cc555b5acdc7aa532e0bea7ef 100644 (file)
@@ -192,7 +192,7 @@ parse_options(int argc, char *argv[])
     error = cfg_set_file(config_file);
     if (error) {
         ofp_fatal(error, "failed to add configuration file \"%s\"", 
-                optarg);
+                config_file);
     }
 }