/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: 2008-09-01 16:19:32 UTC
  • Revision ID: teddy@fukt.bsnet.se-20080901161932-ostp7tulh9aijulh
* plugin-runner.c (add_environment): Never insert existing environment
                                     variables.
  (main): Rename "--global-envs" to "--global-env" and "--envs-for" to
          "--env-for".

* plugin-runner.xml (SYNOPSIS): Rename "--global-envs" to
                                "--global-env" and "--envs-for" to
                                "--env-for".
  (OPTIONS): Added "--global-env" and "--env-for".
  (FALLBACK): Add id attribute.
  (EXIT STATUS): Add text.
  (ENVIRONMENT): New section.
  (FILES): Document configuration file.

Show diffs side-by-side

added added

removed removed

Lines of Context:
plugin-runner.c
96
96
 
97
97
static plugin *plugin_list = NULL;
98
98
 
99
 
/* Gets an existing plugin based on name,
 
99
/* Gets a existing plugin based on name,
100
100
   or if none is found, creates a new one */
101
101
static plugin *getplugin(char *name){
102
102
  /* Check for exiting plugin with that name */
118
118
      return NULL;
119
119
    }
120
120
  }
121
 
  
 
121
 
122
122
  *new_plugin = (plugin) { .name = copy_name,
123
123
                           .argc = 1,
124
124
                           .disabled = false,
179
179
}
180
180
 
181
181
/* Add to a plugin's environment */
182
 
static bool add_environment(plugin *p, const char *def, bool replace){
 
182
static bool add_environment(plugin *p, const char *def){
183
183
  if(p == NULL){
184
184
    return false;
185
185
  }
187
187
  size_t namelen = (size_t)(strchrnul(def, '=') - def);
188
188
  /* Search for this environment variable */
189
189
  for(char **e = p->environ; *e != NULL; e++){
190
 
    if(strncmp(*e, def, namelen + 1) == 0){
191
 
      /* It already exists */
192
 
      if(replace){
193
 
        char *new = realloc(*e, strlen(def) + 1);
194
 
        if(new == NULL){
195
 
          return false;
196
 
        }
197
 
        *e = new;
198
 
        strcpy(*e, def);
199
 
      }
 
190
    if(strncmp(*e, def, namelen+1) == 0){
 
191
      /* Refuse to add an existing variable */
200
192
      return true;
201
193
    }
202
194
  }
238
230
      /* No child processes */
239
231
      break;
240
232
    }
241
 
    
 
233
 
242
234
    /* A child exited, find it in process_list */
243
235
    while(proc != NULL and proc->pid != pid){
244
236
      proc = proc->next;
255
247
/* Prints out a password to stdout */
256
248
bool print_out_password(const char *buffer, size_t length){
257
249
  ssize_t ret;
 
250
  if(length>0 and buffer[length-1] == '\n'){
 
251
    length--;
 
252
  }
258
253
  for(size_t written = 0; written < length; written += (size_t)ret){
259
254
    ret = TEMP_FAILURE_RETRY(write(STDOUT_FILENO, buffer + written,
260
255
                                   length - written));
341
336
    { .name = "global-options", .key = 'g',
342
337
      .arg = "OPTION[,OPTION[,...]]",
343
338
      .doc = "Options passed to all plugins" },
344
 
    { .name = "global-env", .key = 'G',
 
339
    { .name = "global-env", .key = 'e',
345
340
      .arg = "VAR=value",
346
341
      .doc = "Environment variable passed to all plugins" },
347
342
    { .name = "options-for", .key = 'o',
348
343
      .arg = "PLUGIN:OPTION[,OPTION[,...]]",
349
344
      .doc = "Options passed only to specified plugin" },
350
 
    { .name = "env-for", .key = 'E',
 
345
    { .name = "env-for", .key = 'f',
351
346
      .arg = "PLUGIN:ENV=value",
352
347
      .doc = "Environment variable passed to specified plugin" },
353
348
    { .name = "disable", .key = 'd',
354
349
      .arg = "PLUGIN",
355
350
      .doc = "Disable a specific plugin", .group = 1 },
356
 
    { .name = "enable", .key = 'e',
357
 
      .arg = "PLUGIN",
358
 
      .doc = "Enable a specific plugin", .group = 1 },
359
351
    { .name = "plugin-dir", .key = 128,
360
352
      .arg = "DIRECTORY",
361
353
      .doc = "Specify a different plugin directory", .group = 2 },
375
367
  
376
368
  error_t parse_opt (int key, char *arg, __attribute__((unused))
377
369
                     struct argp_state *state) {
 
370
    /* Get the INPUT argument from `argp_parse', which we know is a
 
371
       pointer to our plugin list pointer. */
378
372
    switch (key) {
379
373
    case 'g':                   /* --global-options */
380
374
      if (arg != NULL){
390
384
        }
391
385
      }
392
386
      break;
393
 
    case 'G':                   /* --global-env */
 
387
    case 'e':                   /* --global-env */
394
388
      if(arg == NULL){
395
389
        break;
396
390
      }
399
393
        if(envdef == NULL){
400
394
          break;
401
395
        }
402
 
        if(not add_environment(getplugin(NULL), envdef, true)){
 
396
        if(not add_environment(getplugin(NULL), envdef)){
403
397
          perror("add_environment");
404
398
        }
405
399
      }
407
401
    case 'o':                   /* --options-for */
408
402
      if (arg != NULL){
409
403
        char *p_name = strsep(&arg, ":");
410
 
        if(p_name[0] == '\0' or arg == NULL){
 
404
        if(p_name[0] == '\0'){
411
405
          break;
412
406
        }
413
407
        char *opt = strsep(&arg, ":");
414
 
        if(opt[0] == '\0' or opt == NULL){
 
408
        if(opt[0] == '\0'){
415
409
          break;
416
410
        }
417
 
        char *p;
418
 
        while((p = strsep(&opt, ",")) != NULL){
419
 
          if(p[0] == '\0'){
420
 
            continue;
421
 
          }
422
 
          if(not add_argument(getplugin(p_name), p)){
423
 
            perror("add_argument");
424
 
            return ARGP_ERR_UNKNOWN;
 
411
        if(opt != NULL){
 
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)){
 
418
              perror("add_argument");
 
419
              return ARGP_ERR_UNKNOWN;
 
420
            }
425
421
          }
426
422
        }
427
423
      }
428
424
      break;
429
 
    case 'E':                   /* --env-for */
 
425
    case 'f':                   /* --env-for */
430
426
      if(arg == NULL){
431
427
        break;
432
428
      }
440
436
          break;
441
437
        }
442
438
        envdef++;
443
 
        if(not add_environment(getplugin(p_name), envdef, true)){
 
439
        if(not add_environment(getplugin(p_name), envdef)){
444
440
          perror("add_environment");
445
441
        }
446
442
      }
454
450
        p->disabled = true;
455
451
      }
456
452
      break;
457
 
    case 'e':                   /* --enable */
458
 
      if (arg != NULL){
459
 
        plugin *p = getplugin(arg);
460
 
        if(p == NULL){
461
 
          return ARGP_ERR_UNKNOWN;
462
 
        }
463
 
        p->disabled = false;
464
 
      }
465
 
      break;
466
453
    case 128:                   /* --plugin-dir */
467
454
      plugindir = strdup(arg);
468
455
      if(plugindir == NULL){
470
457
      }      
471
458
      break;
472
459
    case 129:                   /* --config-file */
473
 
      /* This is already done by parse_opt_config_file() */
474
 
      break;
 
460
      argfile = strdup(arg);
 
461
      if(argfile == NULL){
 
462
        perror("strdup");
 
463
      }
 
464
      break;      
475
465
    case 130:                   /* --userid */
476
466
      uid = (uid_t)strtol(arg, NULL, 10);
477
467
      break;
492
482
    return 0;
493
483
  }
494
484
  
495
 
  /* This option parser is the same as parse_opt() above, except it
496
 
     ignores everything but the --config-file option. */
497
 
  error_t parse_opt_config_file (int key, char *arg,
498
 
                                 __attribute__((unused))
499
 
                                 struct argp_state *state) {
500
 
    switch (key) {
501
 
    case 'g':                   /* --global-options */
502
 
    case 'G':                   /* --global-env */
503
 
    case 'o':                   /* --options-for */
504
 
    case 'E':                   /* --env-for */
505
 
    case 'd':                   /* --disable */
506
 
    case 'e':                   /* --enable */
507
 
    case 128:                   /* --plugin-dir */
508
 
      break;
509
 
    case 129:                   /* --config-file */
510
 
      argfile = strdup(arg);
511
 
      if(argfile == NULL){
512
 
        perror("strdup");
513
 
      }
514
 
      break;      
515
 
    case 130:                   /* --userid */
516
 
    case 131:                   /* --groupid */
517
 
    case 132:                   /* --debug */
518
 
    case ARGP_KEY_ARG:
519
 
    case ARGP_KEY_END:
520
 
      break;
521
 
    default:
522
 
      return ARGP_ERR_UNKNOWN;
523
 
    }
524
 
    return 0;
525
 
  }
526
 
  
527
 
  struct argp argp = { .options = options,
528
 
                       .parser = parse_opt_config_file,
529
 
                       .args_doc = "",
 
485
  struct argp argp = { .options = options, .parser = parse_opt,
 
486
                       .args_doc = "[+PLUS_SEPARATED_OPTIONS]",
530
487
                       .doc = "Mandos plugin runner -- Run plugins" };
531
488
  
532
 
  /* Parse using the parse_opt_config_file in order to get the custom
533
 
     config file location, if any. */
534
 
  ret = argp_parse (&argp, argc, argv, ARGP_IN_ORDER, 0, NULL);
 
489
  ret = argp_parse (&argp, argc, argv, 0, 0, NULL);
535
490
  if (ret == ARGP_ERR_UNKNOWN){
536
491
    fprintf(stderr, "Unknown error while parsing arguments\n");
537
492
    exitstatus = EXIT_FAILURE;
538
493
    goto fallback;
539
494
  }
540
 
  
541
 
  /* Reset to the normal argument parser */
542
 
  argp.parser = parse_opt;
543
 
  
544
 
  /* Open the configfile if available */
 
495
 
 
496
  /* Opens the configfile if aviable */
545
497
  if (argfile == NULL){
546
498
    conffp = fopen(AFILE, "r");
547
499
  } else {
601
553
      }
602
554
    }
603
555
    free(org_line);
604
 
  } else {
 
556
  } else{
605
557
    /* Check for harmful errors and go to fallback. Other errors might
606
558
       not affect opening plugins */
607
559
    if (errno == EMFILE or errno == ENFILE or errno == ENOMEM){
613
565
  /* If there was any arguments from configuration file,
614
566
     pass them to parser as command arguments */
615
567
  if(custom_argv != NULL){
616
 
    ret = argp_parse (&argp, custom_argc, custom_argv, ARGP_IN_ORDER,
617
 
                      0, NULL);
 
568
    ret = argp_parse (&argp, custom_argc, custom_argv, 0, 0, NULL);
618
569
    if (ret == ARGP_ERR_UNKNOWN){
619
570
      fprintf(stderr, "Unknown error while parsing arguments\n");
620
571
      exitstatus = EXIT_FAILURE;
622
573
    }
623
574
  }
624
575
  
625
 
  /* Parse actual command line arguments, to let them override the
626
 
     config file */
627
 
  ret = argp_parse (&argp, argc, argv, ARGP_IN_ORDER, 0, NULL);
628
 
  if (ret == ARGP_ERR_UNKNOWN){
629
 
    fprintf(stderr, "Unknown error while parsing arguments\n");
630
 
    exitstatus = EXIT_FAILURE;
631
 
    goto fallback;
632
 
  }
633
 
  
634
576
  if(debug){
635
577
    for(plugin *p = plugin_list; p != NULL; p=p->next){
636
578
      fprintf(stderr, "Plugin: %s has %d arguments\n",
644
586
      }
645
587
    }
646
588
  }
647
 
  
 
589
 
648
590
  /* Strip permissions down to nobody */
649
591
  ret = setuid(uid);
650
592
  if (ret == -1){
654
596
  if (ret == -1){
655
597
    perror("setgid");
656
598
  }
657
 
  
 
599
 
658
600
  if (plugindir == NULL){
659
601
    dir = opendir(PDIR);
660
602
  } else {
681
623
  }
682
624
  
683
625
  FD_ZERO(&rfds_all);
684
 
  
 
626
 
685
627
  /* Read and execute any executable in the plugin directory*/
686
628
  while(true){
687
629
    dirst = readdir(dir);
688
630
    
689
 
    /* All directory entries have been processed */
 
631
    // All directory entries have been processed
690
632
    if(dirst == NULL){
691
633
      if (errno == EBADF){
692
634
        perror("readdir");
698
640
    
699
641
    d_name_len = strlen(dirst->d_name);
700
642
    
701
 
    /* Ignore dotfiles, backup files and other junk */
 
643
    // Ignore dotfiles, backup files and other junk
702
644
    {
703
645
      bool bad_name = false;
704
646
      
790
732
        }
791
733
        /* Add global environment variables */
792
734
        for(char **e = g->environ; *e != NULL; e++){
793
 
          if(not add_environment(p, *e, false)){
 
735
          if(not add_environment(p, *e)){
794
736
            perror("add_environment");
795
737
          }
796
738
        }
801
743
       process, too. */
802
744
    if(p->environ[0] != NULL){
803
745
      for(char **e = environ; *e != NULL; e++){
804
 
        if(not add_environment(p, *e, false)){
 
746
        char *copy = strdup(*e);
 
747
        if(copy == NULL){
 
748
          perror("strdup");
 
749
          continue;
 
750
        }
 
751
        if(not add_environment(p, copy)){
805
752
          perror("add_environment");
806
753
        }
807
754
      }
834
781
      exitstatus = EXIT_FAILURE;
835
782
      goto fallback;
836
783
    }
837
 
    /* Starting a new process to be watched */
 
784
    // Starting a new process to be watched
838
785
    pid_t pid = fork();
839
786
    if(pid == -1){
840
787
      perror("fork");
1038
985
    bool bret;
1039
986
    fprintf(stderr, "Going to fallback mode using getpass(3)\n");
1040
987
    char *passwordbuffer = getpass("Password: ");
1041
 
    size_t len = strlen(passwordbuffer);
1042
 
    /* Strip trailing newline */
1043
 
    if(len > 0 and passwordbuffer[len-1] == '\n'){
1044
 
      passwordbuffer[len-1] = '\0'; /* not strictly necessary */
1045
 
      len--;
1046
 
    }
1047
 
    bret = print_out_password(passwordbuffer, len);
 
988
    bret = print_out_password(passwordbuffer, strlen(passwordbuffer));
1048
989
    if(not bret){
1049
990
      perror("print_out_password");
1050
991
      exitstatus = EXIT_FAILURE;