/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

  • Committer: Teddy Hogeborn
  • Date: 2008-09-01 16:53:17 UTC
  • Revision ID: teddy@fukt.bsnet.se-20080901165317-77dccp81zyqimt2j
* plugin-runner.c (add_environment): Override existing environment
                                     variables when asked to do so.
                                     All callers changed.
  (main): Removed old argp.args_doc string.  Parse argument file
          before normal command line arguments.  Use ARGP_IN_ORDER
          flag in both calls to argp_parse.  Do not strdup() strings
          to be sent to add_environment().

* plugin-runner.xml (OPTIONS): Document "--global-env" and "--envs-for".
  (FILES): Note that the config file is parsed before the command line
           options.

Show diffs side-by-side

added added

removed removed

Lines of Context:
plugin-runner.c
344
344
    { .name = "global-options", .key = 'g',
345
345
      .arg = "OPTION[,OPTION[,...]]",
346
346
      .doc = "Options passed to all plugins" },
347
 
    { .name = "global-env", .key = 'G',
 
347
    { .name = "global-env", .key = 'e',
348
348
      .arg = "VAR=value",
349
349
      .doc = "Environment variable passed to all plugins" },
350
350
    { .name = "options-for", .key = 'o',
351
351
      .arg = "PLUGIN:OPTION[,OPTION[,...]]",
352
352
      .doc = "Options passed only to specified plugin" },
353
 
    { .name = "env-for", .key = 'E',
 
353
    { .name = "env-for", .key = 'f',
354
354
      .arg = "PLUGIN:ENV=value",
355
355
      .doc = "Environment variable passed to specified plugin" },
356
356
    { .name = "disable", .key = 'd',
357
357
      .arg = "PLUGIN",
358
358
      .doc = "Disable a specific plugin", .group = 1 },
359
 
    { .name = "enable", .key = 'e',
360
 
      .arg = "PLUGIN",
361
 
      .doc = "Enable a specific plugin", .group = 1 },
362
359
    { .name = "plugin-dir", .key = 128,
363
360
      .arg = "DIRECTORY",
364
361
      .doc = "Specify a different plugin directory", .group = 2 },
378
375
  
379
376
  error_t parse_opt (int key, char *arg, __attribute__((unused))
380
377
                     struct argp_state *state) {
 
378
    /* Get the INPUT argument from `argp_parse', which we know is a
 
379
       pointer to our plugin list pointer. */
381
380
    switch (key) {
382
381
    case 'g':                   /* --global-options */
383
382
      if (arg != NULL){
393
392
        }
394
393
      }
395
394
      break;
396
 
    case 'G':                   /* --global-env */
 
395
    case 'e':                   /* --global-env */
397
396
      if(arg == NULL){
398
397
        break;
399
398
      }
431
430
        }
432
431
      }
433
432
      break;
434
 
    case 'E':                   /* --env-for */
 
433
    case 'f':                   /* --env-for */
435
434
      if(arg == NULL){
436
435
        break;
437
436
      }
459
458
        p->disabled = true;
460
459
      }
461
460
      break;
462
 
    case 'e':                   /* --enable */
463
 
      if (arg != NULL){
464
 
        plugin *p = getplugin(arg);
465
 
        if(p == NULL){
466
 
          return ARGP_ERR_UNKNOWN;
467
 
        }
468
 
        p->disabled = false;
469
 
      }
470
 
      break;
471
461
    case 128:                   /* --plugin-dir */
472
462
      plugindir = strdup(arg);
473
463
      if(plugindir == NULL){
475
465
      }      
476
466
      break;
477
467
    case 129:                   /* --config-file */
478
 
      /* This is already done by parse_opt_config_file() */
479
 
      break;
 
468
      argfile = strdup(arg);
 
469
      if(argfile == NULL){
 
470
        perror("strdup");
 
471
      }
 
472
      break;      
480
473
    case 130:                   /* --userid */
481
474
      uid = (uid_t)strtol(arg, NULL, 10);
482
475
      break;
497
490
    return 0;
498
491
  }
499
492
  
500
 
  /* This option parser is the same as parse_opt() above, except it
501
 
     ignores everything but the --config-file option. */
502
 
  error_t parse_opt_config_file (int key, char *arg,
503
 
                                 __attribute__((unused))
504
 
                                 struct argp_state *state) {
505
 
    switch (key) {
506
 
    case 'g':                   /* --global-options */
507
 
    case 'G':                   /* --global-env */
508
 
    case 'o':                   /* --options-for */
509
 
    case 'E':                   /* --env-for */
510
 
    case 'd':                   /* --disable */
511
 
    case 'e':                   /* --enable */
512
 
    case 128:                   /* --plugin-dir */
513
 
      break;
514
 
    case 129:                   /* --config-file */
515
 
      argfile = strdup(arg);
516
 
      if(argfile == NULL){
517
 
        perror("strdup");
518
 
      }
519
 
      break;      
520
 
    case 130:                   /* --userid */
521
 
    case 131:                   /* --groupid */
522
 
    case 132:                   /* --debug */
523
 
    case ARGP_KEY_ARG:
524
 
    case ARGP_KEY_END:
525
 
      break;
526
 
    default:
527
 
      return ARGP_ERR_UNKNOWN;
528
 
    }
529
 
    return 0;
530
 
  }
531
 
  
532
 
  struct argp argp = { .options = options,
533
 
                       .parser = parse_opt_config_file,
 
493
  struct argp argp = { .options = options, .parser = parse_opt,
534
494
                       .args_doc = "",
535
495
                       .doc = "Mandos plugin runner -- Run plugins" };
536
496
  
537
 
  /* Parse using the parse_opt_config_file in order to get the custom
538
 
     config file location, if any. */
539
 
  ret = argp_parse (&argp, argc, argv, ARGP_IN_ORDER, 0, NULL);
540
 
  if (ret == ARGP_ERR_UNKNOWN){
541
 
    fprintf(stderr, "Unknown error while parsing arguments\n");
542
 
    exitstatus = EXIT_FAILURE;
543
 
    goto fallback;
544
 
  }
545
 
  
546
 
  /* Reset to the normal argument parser */
547
 
  argp.parser = parse_opt;
548
 
  
549
497
  /* Open the configfile if available */
550
498
  if (argfile == NULL){
551
499
    conffp = fopen(AFILE, "r");