/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-17 00:34:09 UTC
  • Revision ID: teddy@fukt.bsnet.se-20080917003409-bxbjsc4o69nx40t6
* .bzr-builddeb/default.conf: New.

* Makefile (install-server): Do not create directories that
                             should already be present in a
                             normal system.
  (install-client-nokey): - '' -  Also specify non-executable
                          mode for "initramfs-tools-hook-conf".

* README: Improved wording.  Use real copyright mark.

* debian/changelog: New.
* debian/compat: - '' -
* debian/control: - '' -
* debian/copyright: - '' -
* debian/mandos-client.README.Debian: - '' -
* debian/mandos-client.dirs: - '' -
* debian/mandos-client.lintian-overrides: - '' -
* debian/mandos-client.postinst: - '' -
* debian/mandos-client.postrm: - '' -
* debian/mandos.dirs: - '' -
* debian/mandos.lintian-overrides: - '' -
* debian/rules: - '' -
* default-mandos: - '' -

Show diffs side-by-side

added added

removed removed

Lines of Context:
plugin-runner.c
394
394
      if(arg == NULL){
395
395
        break;
396
396
      }
397
 
      {
398
 
        char *envdef = strdup(arg);
399
 
        if(envdef == NULL){
400
 
          break;
401
 
        }
402
 
        if(not add_environment(getplugin(NULL), envdef, true)){
403
 
          perror("add_environment");
404
 
        }
 
397
      if(not add_environment(getplugin(NULL), arg, true)){
 
398
        perror("add_environment");
405
399
      }
406
400
      break;
407
401
    case 'o':                   /* --options-for */
435
429
        if(envdef == NULL){
436
430
          break;
437
431
        }
438
 
        char *p_name = strndup(arg, (size_t) (envdef-arg));
439
 
        if(p_name == NULL){
440
 
          break;
441
 
        }
442
 
        envdef++;
443
 
        if(not add_environment(getplugin(p_name), envdef, true)){
 
432
        *envdef = '\0';
 
433
        if(not add_environment(getplugin(arg), envdef+1, true)){
444
434
          perror("add_environment");
445
435
        }
446
436
      }
464
454
      }
465
455
      break;
466
456
    case 128:                   /* --plugin-dir */
 
457
      free(plugindir);
467
458
      plugindir = strdup(arg);
468
459
      if(plugindir == NULL){
469
460
        perror("strdup");
482
473
      debug = true;
483
474
      break;
484
475
    case ARGP_KEY_ARG:
485
 
      fprintf(stderr, "Ignoring unknown argument \"%s\"\n", arg);
 
476
      /* Cryptsetup always passes an argument, which is an empty
 
477
         string if "none" was specified in /etc/crypttab.  So if
 
478
         argument was empty, we ignore it silently. */
 
479
      if(arg[0] != '\0'){
 
480
        fprintf(stderr, "Ignoring unknown argument \"%s\"\n", arg);
 
481
      }
486
482
      break;
487
483
    case ARGP_KEY_END:
488
484
      break;
507
503
    case 128:                   /* --plugin-dir */
508
504
      break;
509
505
    case 129:                   /* --config-file */
 
506
      free(argfile);
510
507
      argfile = strdup(arg);
511
508
      if(argfile == NULL){
512
509
        perror("strdup");
742
739
    }
743
740
 
744
741
    char *filename;
745
 
    ret = asprintf(&filename, "%s/%s", plugindir, dirst->d_name);
 
742
    if(plugindir == NULL){
 
743
      ret = asprintf(&filename, PDIR "/%s", dirst->d_name);
 
744
    } else {
 
745
      ret = asprintf(&filename, "%s/%s", plugindir, dirst->d_name);
 
746
    }
746
747
    if(ret < 0){
747
748
      perror("asprintf");
748
749
      continue;
937
938
    }
938
939
    /* OK, now either a process completed, or something can be read
939
940
       from one of them */
940
 
    for(plugin *proc = plugin_list; proc != NULL; proc = proc->next){
 
941
    for(plugin *proc = plugin_list; proc != NULL;){
941
942
      /* Is this process completely done? */
942
943
      if(proc->eof and proc->completed){
943
944
        /* Only accept the plugin output if it exited cleanly */
970
971
            exitstatus = EXIT_FAILURE;
971
972
            goto fallback;
972
973
          }
973
 
          free_plugin(proc);
 
974
          
974
975
          /* We are done modifying process list, so unblock signal */
975
976
          ret = sigprocmask (SIG_UNBLOCK, &sigchld_action.sa_mask,
976
977
                             NULL);
983
984
          if(plugin_list == NULL){
984
985
            break;
985
986
          }
 
987
          
 
988
          plugin *next_plugin = proc->next;
 
989
          free_plugin(proc);
 
990
          proc = next_plugin;
986
991
          continue;
987
992
        }
988
993
        
1000
1005
      /* This process has not completed.  Does it have any output? */
1001
1006
      if(proc->eof or not FD_ISSET(proc->fd, &rfds)){
1002
1007
        /* This process had nothing to say at this time */
 
1008
        proc = proc->next;
1003
1009
        continue;
1004
1010
      }
1005
1011
      /* Before reading, make the process' data buffer large enough */
1018
1024
                 BUFFER_SIZE);
1019
1025
      if(ret < 0){
1020
1026
        /* Read error from this process; ignore the error */
 
1027
        proc = proc->next;
1021
1028
        continue;
1022
1029
      }
1023
1030
      if(ret == 0){