/mandos/release

To get this branch, use:
bzr branch http://bzr.recompile.se/loggerhead/mandos/release

« back to all changes in this revision

Viewing changes to plugins.d/mandos-client.c

Convert some programs to use the exit codes from <sysexits.h>.  Change
all programs using the "argp" parsing functions to use them correctly;
checking return value, using argp_error() to report parse errors etc.

* plugin-runner.c: Use <sysexits.h> exit codes.  Always use fallback,
                   even on option errors, except for "--help", etc.
  (getplugin): Make sure "errno" is set correctly on return.
  (main): Declare our own "--help", "--usage", and "--version"
          options which do not cause the fallback to be invoked.
          In all other options, use fallback on any error.
  (parse_opt, parse_opt_config_file): Reset errno at start and return
                                      errno.  No need to check "arg"
                                      for NULL.  New "--help",
                                      "--usage", and "--version"
                                      options.
  (parse_opt): Accept empty string as global option.  Do not print
               errors which will be detected and reported later.  Do
               "argp_error()" on parse error or empty plugin names.
* plugins.d/mandos-client.c: Use <sysexits.h> exit codes.  Do not
                             return successful exit code on "--help",
                             etc. since this would give the wrong
                             message to "plugin-runner".
  (main): Declare our own "--help", "--usage", and "--version"
          options which do not return a successful exit code.
  (parse_opt): Reset errno at start and return errno.  Do
               "argp_error()" on parse errors.  New "--help",
               "--usage", and "--version" options.
* plugins.d/password-prompt.c: Use exit codes from <sysexits.h>.  Do
                               not return successful exit code on
                               "--help", etc. since this would give
                               the wrong message to "plugin-runner".
  (main): Declare our own "--help", "--usage", and "--version" options
          which do not return a successful exit code.  Do
          close(STDOUT_FILENO) after writing to check its return code.
  (parse_opt): Reset errno at start and return errno.

Show diffs side-by-side

added added

removed removed

Lines of Context:
plugins.d/mandos-client.c
82
82
#include <signal.h>             /* sigemptyset(), sigaddset(),
83
83
                                   sigaction(), SIGTERM, sig_atomic_t,
84
84
                                   raise() */
 
85
#include <sysexits.h>           /* EX_OSERR, EX_USAGE */
85
86
 
86
87
#ifdef __linux__
87
88
#include <sys/klog.h>           /* klogctl() */
1056
1057
        .arg = "SECONDS",
1057
1058
        .doc = "Maximum delay to wait for interface startup",
1058
1059
        .group = 2 },
 
1060
      /*
 
1061
       * These reproduce what we would get without ARGP_NO_HELP
 
1062
       */
 
1063
      { .name = "help", .key = '?',
 
1064
        .doc = "Give this help list", .group = -1 },
 
1065
      { .name = "usage", .key = -3,
 
1066
        .doc = "Give a short usage message", .group = -1 },
 
1067
      { .name = "version", .key = 'V',
 
1068
        .doc = "Print program version", .group = -1 },
1059
1069
      { .name = NULL }
1060
1070
    };
1061
1071
    
1062
1072
    error_t parse_opt(int key, char *arg,
1063
1073
                      struct argp_state *state){
 
1074
      errno = 0;
1064
1075
      switch(key){
1065
1076
      case 128:                 /* --debug */
1066
1077
        debug = true;
1082
1093
        tmpmax = strtoimax(arg, &tmp, 10);
1083
1094
        if(errno != 0 or tmp == arg or *tmp != '\0'
1084
1095
           or tmpmax != (typeof(mc.dh_bits))tmpmax){
1085
 
          fprintf(stderr, "Bad number of DH bits\n");
1086
 
          exit(EXIT_FAILURE);
 
1096
          argp_error(state, "Bad number of DH bits");
1087
1097
        }
1088
1098
        mc.dh_bits = (typeof(mc.dh_bits))tmpmax;
1089
1099
        break;
1094
1104
        errno = 0;
1095
1105
        delay = strtof(arg, &tmp);
1096
1106
        if(errno != 0 or tmp == arg or *tmp != '\0'){
1097
 
          fprintf(stderr, "Bad delay\n");
1098
 
          exit(EXIT_FAILURE);
 
1107
          argp_error(state, "Bad delay");
1099
1108
        }
1100
1109
        break;
1101
 
      case ARGP_KEY_ARG:
1102
 
        argp_usage(state);
1103
 
      case ARGP_KEY_END:
 
1110
        /*
 
1111
         * These reproduce what we would get without ARGP_NO_HELP
 
1112
         */
 
1113
      case '?':                 /* --help */
 
1114
        argp_state_help(state, state->out_stream,
 
1115
                        (ARGP_HELP_STD_HELP | ARGP_HELP_EXIT_ERR)
 
1116
                        & ~(unsigned int)ARGP_HELP_EXIT_OK);
 
1117
      case -3:                  /* --usage */
 
1118
        argp_state_help(state, state->out_stream,
 
1119
                        ARGP_HELP_USAGE | ARGP_HELP_EXIT_ERR);
 
1120
      case 'V':                 /* --version */
 
1121
        fprintf(state->out_stream, "%s\n", argp_program_version);
 
1122
        exit(argp_err_exit_status);
1104
1123
        break;
1105
1124
      default:
1106
1125
        return ARGP_ERR_UNKNOWN;
1107
1126
      }
1108
 
      return 0;
 
1127
      return errno;
1109
1128
    }
1110
1129
    
1111
1130
    struct argp argp = { .options = options, .parser = parse_opt,
1112
1131
                         .args_doc = "",
1113
1132
                         .doc = "Mandos client -- Get and decrypt"
1114
1133
                         " passwords from a Mandos server" };
1115
 
    ret = argp_parse(&argp, argc, argv, 0, 0, NULL);
1116
 
    if(ret == ARGP_ERR_UNKNOWN){
1117
 
      fprintf(stderr, "Unknown error while parsing arguments\n");
1118
 
      exitcode = EXIT_FAILURE;
 
1134
    ret = argp_parse(&argp, argc, argv,
 
1135
                     ARGP_IN_ORDER | ARGP_NO_HELP, 0, NULL);
 
1136
    switch(ret){
 
1137
    case 0:
 
1138
      break;
 
1139
    case ENOMEM:
 
1140
    default:
 
1141
      errno = ret;
 
1142
      perror("argp_parse");
 
1143
      exitcode = EX_OSERR;
 
1144
      goto end;
 
1145
    case EINVAL:
 
1146
      exitcode = EX_USAGE;
1119
1147
      goto end;
1120
1148
    }
1121
1149
  }
1220
1248
    
1221
1249
#ifdef __linux__
1222
1250
    /* Lower kernel loglevel to KERN_NOTICE to avoid KERN_INFO
1223
 
       messages to mess up the prompt */
 
1251
       messages about the network interface to mess up the prompt */
1224
1252
    ret = klogctl(8, NULL, 5);
1225
1253
    bool restore_loglevel = true;
1226
1254
    if(ret == -1){
1530
1558
      if(ret == -1){
1531
1559
        perror("ioctl SIOCGIFFLAGS");
1532
1560
      } else if(network.ifr_flags & IFF_UP) {
1533
 
        network.ifr_flags &= ~IFF_UP; /* clear flag */
 
1561
        network.ifr_flags &= ~(short)IFF_UP; /* clear flag */
1534
1562
        ret = ioctl(sd, SIOCSIFFLAGS, &network);
1535
1563
        if(ret == -1){
1536
1564
          perror("ioctl SIOCSIFFLAGS");