/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 plugin-runner.c

Fixes for sscanf usage:

* plugin-runner.c (main): Parse numbers correctly and portably using
                          an intermediate "intmax_t".  Cast "pid_t" to
                          "intmax_t" before passing it to "printf()".
* plugins.d/mandos-client.c (main): Parse numbers correctly and
                                    portably using an intermediate
                                    "intmax_t".  Bug fix: cast
                                    "AvahiIfIndex" to "intmax_t" and
                                    use "PRIdMAX" instead of using
                                    "PRIu16", and use "PRIu16" to
                                    format port number.
* plugins.d/splashy.c (main): Parse numbers correctly and portably
                              using an intermediate "intmax_t".
* plugins.d/usplash.c (main): - '' -

Show diffs side-by-side

added added

removed removed

Lines of Context:
plugin-runner.c
62
62
#include <signal.h>             /* struct sigaction, sigemptyset(),
63
63
                                   sigaddset(), sigaction(),
64
64
                                   sigprocmask(), SIG_BLOCK, SIGCHLD,
65
 
                                   SIG_UNBLOCK, kill(), sig_atomic_t
66
 
                                */
 
65
                                   SIG_UNBLOCK, kill() */
67
66
#include <errno.h>              /* errno, EBADF */
68
 
#include <inttypes.h>           /* intmax_t, PRIdMAX, strtoimax() */
 
67
#include <inttypes.h>           /* intmax_t, SCNdMAX, PRIdMAX,  */
69
68
 
70
69
#define BUFFER_SIZE 256
71
70
 
82
81
  char **environ;
83
82
  int envc;
84
83
  bool disabled;
85
 
  
 
84
 
86
85
  /* Variables used for running processes*/
87
86
  pid_t pid;
88
87
  int fd;
90
89
  size_t buffer_size;
91
90
  size_t buffer_length;
92
91
  bool eof;
93
 
  volatile sig_atomic_t completed;
94
 
  int status;
 
92
  volatile bool completed;
 
93
  volatile int status;
95
94
  struct plugin *next;
96
95
} plugin;
97
96
 
116
115
  if(name != NULL){
117
116
    copy_name = strdup(name);
118
117
    if(copy_name == NULL){
119
 
      free(new_plugin);
120
118
      return NULL;
121
119
    }
122
120
  }
123
121
  
124
 
  *new_plugin = (plugin){ .name = copy_name,
125
 
                          .argc = 1,
126
 
                          .disabled = false,
127
 
                          .next = plugin_list };
 
122
  *new_plugin = (plugin) { .name = copy_name,
 
123
                           .argc = 1,
 
124
                           .disabled = false,
 
125
                           .next = plugin_list };
128
126
  
129
127
  new_plugin->argv = malloc(sizeof(char *) * 2);
130
128
  if(new_plugin->argv == NULL){
224
222
/* Mark processes as completed when they exit, and save their exit
225
223
   status. */
226
224
static void handle_sigchld(__attribute__((unused)) int sig){
227
 
  int old_errno = errno;
228
225
  while(true){
229
226
    plugin *proc = plugin_list;
230
227
    int status;
234
231
      break;
235
232
    }
236
233
    if(pid == -1){
237
 
      if(errno == ECHILD){
238
 
        /* No child processes */
239
 
        break;
 
234
      if(errno != ECHILD){
 
235
        perror("waitpid");
240
236
      }
241
 
      perror("waitpid");
 
237
      /* No child processes */
 
238
      break;
242
239
    }
243
240
    
244
241
    /* A child exited, find it in process_list */
250
247
      continue;
251
248
    }
252
249
    proc->status = status;
253
 
    proc->completed = 1;
 
250
    proc->completed = true;
254
251
  }
255
 
  errno = old_errno;
256
252
}
257
253
 
258
254
/* Prints out a password to stdout */
280
276
  }
281
277
  free(plugin_node->environ);
282
278
  free(plugin_node->buffer);
283
 
  
 
279
 
284
280
  /* Removes the plugin from the singly-linked list */
285
281
  if(plugin_node == plugin_list){
286
282
    /* First one - simple */
313
309
  struct dirent *dirst;
314
310
  struct stat st;
315
311
  fd_set rfds_all;
316
 
  int ret, maxfd = 0;
 
312
  int ret, numchars, maxfd = 0;
317
313
  ssize_t sret;
318
314
  intmax_t tmpmax;
319
315
  uid_t uid = 65534;
379
375
  };
380
376
  
381
377
  error_t parse_opt(int key, char *arg, __attribute__((unused))
382
 
                    struct argp_state *state){
383
 
    char *tmp;
384
 
    switch(key){
 
378
                    struct argp_state *state) {
 
379
    switch(key) {
385
380
    case 'g':                   /* --global-options */
386
381
      if(arg != NULL){
387
 
        char *plugin_option;
388
 
        while((plugin_option = strsep(&arg, ",")) != NULL){
389
 
          if(plugin_option[0] == '\0'){
 
382
        char *p;
 
383
        while((p = strsep(&arg, ",")) != NULL){
 
384
          if(p[0] == '\0'){
390
385
            continue;
391
386
          }
392
 
          if(not add_argument(getplugin(NULL), plugin_option)){
 
387
          if(not add_argument(getplugin(NULL), p)){
393
388
            perror("add_argument");
394
389
            return ARGP_ERR_UNKNOWN;
395
390
          }
406
401
      break;
407
402
    case 'o':                   /* --options-for */
408
403
      if(arg != NULL){
409
 
        char *plugin_name = strsep(&arg, ":");
410
 
        if(plugin_name[0] == '\0'){
411
 
          break;
412
 
        }
413
 
        char *plugin_option;
414
 
        while((plugin_option = strsep(&arg, ",")) != NULL){
415
 
          if(not add_argument(getplugin(plugin_name), plugin_option)){
 
404
        char *p_name = strsep(&arg, ":");
 
405
        if(p_name[0] == '\0' or arg == NULL){
 
406
          break;
 
407
        }
 
408
        char *opt = strsep(&arg, ":");
 
409
        if(opt[0] == '\0' or opt == NULL){
 
410
          break;
 
411
        }
 
412
        char *p;
 
413
        while((p = strsep(&opt, ",")) != NULL){
 
414
          if(p[0] == '\0'){
 
415
            continue;
 
416
          }
 
417
          if(not add_argument(getplugin(p_name), p)){
416
418
            perror("add_argument");
417
419
            return ARGP_ERR_UNKNOWN;
418
420
          }
463
465
      /* This is already done by parse_opt_config_file() */
464
466
      break;
465
467
    case 130:                   /* --userid */
466
 
      errno = 0;
467
 
      tmpmax = strtoimax(arg, &tmp, 10);
468
 
      if(errno != 0 or tmp == arg or *tmp != '\0'
469
 
         or tmpmax != (uid_t)tmpmax){
 
468
      ret = sscanf(arg, "%" SCNdMAX "%n", &tmpmax, &numchars);
 
469
      if(ret < 1 or tmpmax != (uid_t)tmpmax
 
470
         or arg[numchars] != '\0'){
470
471
        fprintf(stderr, "Bad user ID number: \"%s\", using %"
471
472
                PRIdMAX "\n", arg, (intmax_t)uid);
472
473
      } else {
474
475
      }
475
476
      break;
476
477
    case 131:                   /* --groupid */
477
 
      errno = 0;
478
 
      tmpmax = strtoimax(arg, &tmp, 10);
479
 
      if(errno != 0 or tmp == arg or *tmp != '\0'
480
 
         or tmpmax != (gid_t)tmpmax){
 
478
      ret = sscanf(arg, "%" SCNdMAX "%n", &tmpmax, &numchars);
 
479
      if(ret < 1 or tmpmax != (gid_t)tmpmax
 
480
         or arg[numchars] != '\0'){
481
481
        fprintf(stderr, "Bad group ID number: \"%s\", using %"
482
482
                PRIdMAX "\n", arg, (intmax_t)gid);
483
483
      } else {
511
511
     ignores everything but the --config-file option. */
512
512
  error_t parse_opt_config_file(int key, char *arg,
513
513
                                __attribute__((unused))
514
 
                                struct argp_state *state){
515
 
    switch(key){
 
514
                                struct argp_state *state) {
 
515
    switch(key) {
516
516
    case 'g':                   /* --global-options */
517
517
    case 'G':                   /* --global-env */
518
518
    case 'o':                   /* --options-for */
569
569
    size_t size = 0;
570
570
    const char whitespace_delims[] = " \r\t\f\v\n";
571
571
    const char comment_delim[] = "#";
572
 
    
 
572
 
573
573
    custom_argc = 1;
574
574
    custom_argv = malloc(sizeof(char*) * 2);
575
575
    if(custom_argv == NULL){
579
579
    }
580
580
    custom_argv[0] = argv[0];
581
581
    custom_argv[1] = NULL;
582
 
    
 
582
 
583
583
    /* for each line in the config file, strip whitespace and ignore
584
584
       commented text */
585
585
    while(true){
587
587
      if(sret == -1){
588
588
        break;
589
589
      }
590
 
      
 
590
 
591
591
      line = org_line;
592
592
      arg = strsep(&line, comment_delim);
593
593
      while((p = strsep(&arg, whitespace_delims)) != NULL){
661
661
  }
662
662
  
663
663
  /* Strip permissions down to nobody */
 
664
  ret = setuid(uid);
 
665
  if(ret == -1){
 
666
    perror("setuid");
 
667
  }  
664
668
  setgid(gid);
665
669
  if(ret == -1){
666
670
    perror("setgid");
667
671
  }
668
 
  ret = setuid(uid);
669
 
  if(ret == -1){
670
 
    perror("setuid");
671
 
  }
672
672
  
673
673
  if(plugindir == NULL){
674
674
    dir = opendir(PDIR);
756
756
        continue;
757
757
      }
758
758
    }
759
 
    
 
759
 
760
760
    char *filename;
761
761
    if(plugindir == NULL){
762
762
      ret = asprintf(&filename, PDIR "/%s", dirst->d_name);
774
774
      free(filename);
775
775
      continue;
776
776
    }
777
 
    
 
777
 
778
778
    /* Ignore non-executable files */
779
779
    if(not S_ISREG(st.st_mode) or (access(filename, X_OK) != 0)){
780
780
      if(debug){
933
933
  
934
934
  closedir(dir);
935
935
  dir = NULL;
936
 
  free_plugin(getplugin(NULL));
937
936
  
938
937
  for(plugin *p = plugin_list; p != NULL; p = p->next){
939
938
    if(p->pid != 0){
959
958
       from one of them */
960
959
    for(plugin *proc = plugin_list; proc != NULL;){
961
960
      /* Is this process completely done? */
962
 
      if(proc->completed and proc->eof){
 
961
      if(proc->eof and proc->completed){
963
962
        /* Only accept the plugin output if it exited cleanly */
964
963
        if(not WIFEXITED(proc->status)
965
964
           or WEXITSTATUS(proc->status) != 0){
966
965
          /* Bad exit by plugin */
967
 
          
 
966
 
968
967
          if(debug){
969
968
            if(WIFEXITED(proc->status)){
970
 
              fprintf(stderr, "Plugin %s [%" PRIdMAX "] exited with"
971
 
                      " status %d\n", proc->name,
972
 
                      (intmax_t) (proc->pid),
 
969
              fprintf(stderr, "Plugin %" PRIdMAX " exited with status"
 
970
                      " %d\n", (intmax_t) (proc->pid),
973
971
                      WEXITSTATUS(proc->status));
974
 
            } else if(WIFSIGNALED(proc->status)){
975
 
              fprintf(stderr, "Plugin %s [%" PRIdMAX "] killed by"
976
 
                      " signal %d\n", proc->name,
977
 
                      (intmax_t) (proc->pid),
 
972
            } else if(WIFSIGNALED(proc->status)) {
 
973
              fprintf(stderr, "Plugin %" PRIdMAX " killed by signal"
 
974
                      " %d\n", (intmax_t) (proc->pid),
978
975
                      WTERMSIG(proc->status));
979
976
            } else if(WCOREDUMP(proc->status)){
980
 
              fprintf(stderr, "Plugin %s [%" PRIdMAX "] dumped"
981
 
                      " core\n", proc->name, (intmax_t) (proc->pid));
 
977
              fprintf(stderr, "Plugin %" PRIdMAX " dumped core\n",
 
978
                      (intmax_t) (proc->pid));
982
979
            }
983
980
          }
984
981
          
985
982
          /* Remove the plugin */
986
983
          FD_CLR(proc->fd, &rfds_all);
987
 
          
 
984
 
988
985
          /* Block signal while modifying process_list */
989
986
          ret = sigprocmask(SIG_BLOCK, &sigchld_action.sa_mask, NULL);
990
987
          if(ret < 0){
1057
1054
      }
1058
1055
    }
1059
1056
  }
1060
 
  
1061
 
  
 
1057
 
 
1058
 
1062
1059
 fallback:
1063
1060
  
1064
1061
  if(plugin_list == NULL or exitstatus != EXIT_SUCCESS){