/mandos/trunk

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

« back to all changes in this revision

Viewing changes to plugin-runner.c

  • Committer: Teddy Hogeborn
  • Date: 2009-01-14 14:20:17 UTC
  • Revision ID: teddy@fukt.bsnet.se-20090114142017-84t4kfw56bsftjlo
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
89
89
  size_t buffer_size;
90
90
  size_t buffer_length;
91
91
  bool eof;
92
 
  volatile sig_atomic_t completed;
93
 
  int status;
 
92
  volatile bool completed;
 
93
  volatile int status;
94
94
  struct plugin *next;
95
95
} plugin;
96
96
 
115
115
  if(name != NULL){
116
116
    copy_name = strdup(name);
117
117
    if(copy_name == NULL){
118
 
      free(new_plugin);
119
118
      return NULL;
120
119
    }
121
120
  }
122
121
  
123
 
  *new_plugin = (plugin){ .name = copy_name,
124
 
                          .argc = 1,
125
 
                          .disabled = false,
126
 
                          .next = plugin_list };
 
122
  *new_plugin = (plugin) { .name = copy_name,
 
123
                           .argc = 1,
 
124
                           .disabled = false,
 
125
                           .next = plugin_list };
127
126
  
128
127
  new_plugin->argv = malloc(sizeof(char *) * 2);
129
128
  if(new_plugin->argv == NULL){
223
222
/* Mark processes as completed when they exit, and save their exit
224
223
   status. */
225
224
static void handle_sigchld(__attribute__((unused)) int sig){
226
 
  int old_errno = errno;
227
225
  while(true){
228
226
    plugin *proc = plugin_list;
229
227
    int status;
233
231
      break;
234
232
    }
235
233
    if(pid == -1){
236
 
      if(errno == ECHILD){
237
 
        /* No child processes */
238
 
        break;
 
234
      if(errno != ECHILD){
 
235
        perror("waitpid");
239
236
      }
240
 
      perror("waitpid");
 
237
      /* No child processes */
 
238
      break;
241
239
    }
242
240
    
243
241
    /* A child exited, find it in process_list */
249
247
      continue;
250
248
    }
251
249
    proc->status = status;
252
 
    proc->completed = 1;
 
250
    proc->completed = true;
253
251
  }
254
 
  errno = old_errno;
255
252
}
256
253
 
257
254
/* Prints out a password to stdout */
378
375
  };
379
376
  
380
377
  error_t parse_opt(int key, char *arg, __attribute__((unused))
381
 
                    struct argp_state *state){
382
 
    switch(key){
 
378
                    struct argp_state *state) {
 
379
    switch(key) {
383
380
    case 'g':                   /* --global-options */
384
381
      if(arg != NULL){
385
 
        char *plugin_option;
386
 
        while((plugin_option = strsep(&arg, ",")) != NULL){
387
 
          if(plugin_option[0] == '\0'){
 
382
        char *p;
 
383
        while((p = strsep(&arg, ",")) != NULL){
 
384
          if(p[0] == '\0'){
388
385
            continue;
389
386
          }
390
 
          if(not add_argument(getplugin(NULL), plugin_option)){
 
387
          if(not add_argument(getplugin(NULL), p)){
391
388
            perror("add_argument");
392
389
            return ARGP_ERR_UNKNOWN;
393
390
          }
404
401
      break;
405
402
    case 'o':                   /* --options-for */
406
403
      if(arg != NULL){
407
 
        char *plugin_name = strsep(&arg, ":");
408
 
        if(plugin_name[0] == '\0'){
409
 
          break;
410
 
        }
411
 
        char *plugin_option;
412
 
        while((plugin_option = strsep(&arg, ",")) != NULL){
413
 
          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)){
414
418
            perror("add_argument");
415
419
            return ARGP_ERR_UNKNOWN;
416
420
          }
507
511
     ignores everything but the --config-file option. */
508
512
  error_t parse_opt_config_file(int key, char *arg,
509
513
                                __attribute__((unused))
510
 
                                struct argp_state *state){
511
 
    switch(key){
 
514
                                struct argp_state *state) {
 
515
    switch(key) {
512
516
    case 'g':                   /* --global-options */
513
517
    case 'G':                   /* --global-env */
514
518
    case 'o':                   /* --options-for */
657
661
  }
658
662
  
659
663
  /* Strip permissions down to nobody */
 
664
  ret = setuid(uid);
 
665
  if(ret == -1){
 
666
    perror("setuid");
 
667
  }  
660
668
  setgid(gid);
661
669
  if(ret == -1){
662
670
    perror("setgid");
663
671
  }
664
 
  ret = setuid(uid);
665
 
  if(ret == -1){
666
 
    perror("setuid");
667
 
  }
668
672
  
669
673
  if(plugindir == NULL){
670
674
    dir = opendir(PDIR);
929
933
  
930
934
  closedir(dir);
931
935
  dir = NULL;
932
 
  free_plugin(getplugin(NULL));
933
936
  
934
937
  for(plugin *p = plugin_list; p != NULL; p = p->next){
935
938
    if(p->pid != 0){
955
958
       from one of them */
956
959
    for(plugin *proc = plugin_list; proc != NULL;){
957
960
      /* Is this process completely done? */
958
 
      if(proc->completed and proc->eof){
 
961
      if(proc->eof and proc->completed){
959
962
        /* Only accept the plugin output if it exited cleanly */
960
963
        if(not WIFEXITED(proc->status)
961
964
           or WEXITSTATUS(proc->status) != 0){
963
966
 
964
967
          if(debug){
965
968
            if(WIFEXITED(proc->status)){
966
 
              fprintf(stderr, "Plugin %s [%" PRIdMAX "] exited with"
967
 
                      " status %d\n", proc->name,
968
 
                      (intmax_t) (proc->pid),
 
969
              fprintf(stderr, "Plugin %" PRIdMAX " exited with status"
 
970
                      " %d\n", (intmax_t) (proc->pid),
969
971
                      WEXITSTATUS(proc->status));
970
 
            } else if(WIFSIGNALED(proc->status)){
971
 
              fprintf(stderr, "Plugin %s [%" PRIdMAX "] killed by"
972
 
                      " signal %d\n", proc->name,
973
 
                      (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),
974
975
                      WTERMSIG(proc->status));
975
976
            } else if(WCOREDUMP(proc->status)){
976
 
              fprintf(stderr, "Plugin %s [%" PRIdMAX "] dumped"
977
 
                      " core\n", proc->name, (intmax_t) (proc->pid));
 
977
              fprintf(stderr, "Plugin %" PRIdMAX " dumped core\n",
 
978
                      (intmax_t) (proc->pid));
978
979
            }
979
980
          }
980
981