/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

* plugin-runner.c (main): Bug fix; do not accept a "d" character after
                          user ID or group ID numbers.  Bug fix: use
                          "%u" when printing PID of coredumped plugin,
                          not "%d".
* plugins.d/password-prompt.c (main): Remove comment which was copied
                                      from another program by mistake.
* plugins.d/splashy.c: Only comment changes.
* plugins.d/usplash.c: - '' -

Show diffs side-by-side

added added

removed removed

Lines of Context:
plugin-runner.c
2
2
/*
3
3
 * Mandos plugin runner - Run Mandos plugins
4
4
 *
5
 
 * Copyright © 2008 Teddy Hogeborn & Björn Påhlsson
 
5
 * Copyright © 2008,2009 Teddy Hogeborn
 
6
 * Copyright © 2008,2009 Björn Påhlsson
6
7
 * 
7
8
 * This program is free software: you can redistribute it and/or
8
9
 * modify it under the terms of the GNU General Public License as
69
70
#define PDIR "/lib/mandos/plugins.d"
70
71
#define AFILE "/conf/conf.d/mandos/plugin-runner.conf"
71
72
 
72
 
const char *argp_program_version = "plugin-runner 1.0";
 
73
const char *argp_program_version = "plugin-runner " VERSION;
73
74
const char *argp_program_bug_address = "<mandos@fukt.bsnet.se>";
74
75
 
75
76
typedef struct plugin{
308
309
  struct stat st;
309
310
  fd_set rfds_all;
310
311
  int ret, maxfd = 0;
 
312
  ssize_t sret;
311
313
  uid_t uid = 65534;
312
314
  gid_t gid = 65534;
313
315
  bool debug = false;
461
463
      /* This is already done by parse_opt_config_file() */
462
464
      break;
463
465
    case 130:                   /* --userid */
464
 
      uid = (uid_t)strtol(arg, NULL, 10);
 
466
      /* In the GNU C library, uid_t is always unsigned int */
 
467
      ret = sscanf(arg, "%u", &uid);
 
468
      if(ret != 1){
 
469
        fprintf(stderr, "Bad user ID number: \"%s\", using %u\n", arg,
 
470
                uid);
 
471
      }
465
472
      break;
466
473
    case 131:                   /* --groupid */
467
 
      gid = (gid_t)strtol(arg, NULL, 10);
 
474
      /* In the GNU C library, gid_t is always unsigned int */
 
475
      ret = sscanf(arg, "%u", &gid);
 
476
      if(ret != 1){
 
477
        fprintf(stderr, "Bad group ID number: \"%s\", using %u\n",
 
478
                arg, gid);
 
479
      }
468
480
      break;
469
481
    case 132:                   /* --debug */
470
482
      debug = true;
471
483
      break;
 
484
/*
 
485
 * When adding more options before this line, remember to also add a
 
486
 * "case" to the "parse_opt_config_file" function below.
 
487
 */
472
488
    case ARGP_KEY_ARG:
473
489
      /* Cryptsetup always passes an argument, which is an empty
474
490
         string if "none" was specified in /etc/crypttab.  So if
523
539
                       .args_doc = "",
524
540
                       .doc = "Mandos plugin runner -- Run plugins" };
525
541
  
526
 
  /* Parse using the parse_opt_config_file in order to get the custom
 
542
  /* Parse using parse_opt_config_file() in order to get the custom
527
543
     config file location, if any. */
528
544
  ret = argp_parse (&argp, argc, argv, ARGP_IN_ORDER, 0, NULL);
529
545
  if (ret == ARGP_ERR_UNKNOWN){
545
561
    char *org_line = NULL;
546
562
    char *p, *arg, *new_arg, *line;
547
563
    size_t size = 0;
548
 
    ssize_t sret;
549
564
    const char whitespace_delims[] = " \r\t\f\v\n";
550
565
    const char comment_delim[] = "#";
551
566
 
700
715
      
701
716
      const char const *bad_suffixes[] = { "~", "#", ".dpkg-new",
702
717
                                           ".dpkg-old",
 
718
                                           ".dpkg-bak",
703
719
                                           ".dpkg-divert", NULL };
704
720
      for(const char **pre = bad_prefixes; *pre != NULL; pre++){
705
721
        size_t pre_len = strlen(*pre);
952
968
                      (unsigned int) (proc->pid),
953
969
                      WTERMSIG(proc->status));
954
970
            } else if(WCOREDUMP(proc->status)){
955
 
              fprintf(stderr, "Plugin %d dumped core\n",
 
971
              fprintf(stderr, "Plugin %u dumped core\n",
956
972
                      (unsigned int) (proc->pid));
957
973
            }
958
974
          }
1017
1033
        proc->buffer_size += BUFFER_SIZE;
1018
1034
      }
1019
1035
      /* Read from the process */
1020
 
      ret = read(proc->fd, proc->buffer + proc->buffer_length,
1021
 
                 BUFFER_SIZE);
1022
 
      if(ret < 0){
 
1036
      sret = read(proc->fd, proc->buffer + proc->buffer_length,
 
1037
                  BUFFER_SIZE);
 
1038
      if(sret < 0){
1023
1039
        /* Read error from this process; ignore the error */
1024
1040
        proc = proc->next;
1025
1041
        continue;
1026
1042
      }
1027
 
      if(ret == 0){
 
1043
      if(sret == 0){
1028
1044
        /* got EOF */
1029
1045
        proc->eof = true;
1030
1046
      } else {
1031
 
        proc->buffer_length += (size_t) ret;
 
1047
        proc->buffer_length += (size_t) sret;
1032
1048
      }
1033
1049
    }
1034
1050
  }