staging: usbip: userspace: usbip: modify command failure
authormatt mooney <mfm@muteddisk.com>
Mon, 20 Jun 2011 05:44:39 +0000 (22:44 -0700)
committerGreg Kroah-Hartman <gregkh@suse.de>
Wed, 6 Jul 2011 03:11:34 +0000 (20:11 -0700)
When a bad option is given, display a message stating such and output
usage. When a bad command is given, output command help.

Signed-off-by: matt mooney <mfm@muteddisk.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
drivers/staging/usbip/userspace/src/usbip.c

index cdfe4c2d9cbe0643ddd7989ae8a364b0663bcf46..583b179a11261ac34c333a31ef9f94da5a6b8b0a 100644 (file)
@@ -125,13 +125,13 @@ static int usbip_version(int argc, char *argv[])
        (void) argc;
        (void) argv;
 
-       printf("%s\n", usbip_version_string);
+       printf(PROGNAME " (%s)\n", usbip_version_string);
        return 0;
 }
 
 static int run_command(const struct command *cmd, int argc, char *argv[])
 {
-       dbg("running command: `%s'\n", cmd->name);
+       dbg("running command: `%s'", cmd->name);
        return cmd->fn(argc, argv);
 }
 
@@ -163,8 +163,11 @@ int main(int argc, char *argv[])
                        usbip_use_syslog = 1;
                        openlog("", LOG_PID, LOG_USER);
                        break;
+               case '?':
+                       printf("usbip: invalid option\n");
                default:
-                       goto err_out;
+                       usbip_usage();
+                       goto out;
                }
        }
 
@@ -180,8 +183,8 @@ int main(int argc, char *argv[])
                        }
        }
 
-err_out:
-       usbip_usage();
+       /* invalid command */
+       usbip_help(0, NULL);
 out:
        return (rc > -1 ? EXIT_SUCCESS : EXIT_FAILURE);
 }