[client] fix missing 'break' in switch statement (discovered by Coverity)
authorMichael Rash <mbr@cipherdyne.org>
Wed, 8 May 2013 01:43:38 +0000 (21:43 -0400)
committerMichael Rash <mbr@cipherdyne.org>
Wed, 8 May 2013 01:43:38 +0000 (21:43 -0400)
client/config_init.c

index 6350689..0c9856f 100644 (file)
@@ -1541,12 +1541,14 @@ config_init(fko_cli_options_t *options, int argc, char **argv)
                 options->use_hmac = 1;
                 cli_arg_bitmask |= FWKNOP_CLI_ARG_BM(FWKNOP_CLI_ARG_KEY_HMAC_BASE64);
                 cli_arg_bitmask |= FWKNOP_CLI_ARG_BM(FWKNOP_CLI_ARG_USE_HMAC);
+                break;
             case KEY_HMAC:
                 strlcpy(options->hmac_key, optarg, sizeof(options->hmac_key));
                 options->have_hmac_key = 1;
                 options->use_hmac = 1;
                 cli_arg_bitmask |= FWKNOP_CLI_ARG_BM(FWKNOP_CLI_ARG_KEY_HMAC);
                 cli_arg_bitmask |= FWKNOP_CLI_ARG_BM(FWKNOP_CLI_ARG_USE_HMAC);
+                break;
             case KEY_LEN:
                 options->key_len = strtol_wrapper(optarg, 1,
                         MAX_KEY_LEN, NO_EXIT_UPON_ERR, &is_err);