/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-05 18:37:28 UTC
  • Revision ID: teddy@fukt.bsnet.se-20080905183728-l0m8v3vqa302uwrw
* mandos (main): Use EAFP with pidfile.

Show diffs side-by-side

added added

removed removed

Lines of Context:
plugin-runner.c
397
397
      if(arg == NULL){
398
398
        break;
399
399
      }
400
 
      if(not add_environment(getplugin(NULL), arg, true)){
401
 
        perror("add_environment");
 
400
      {
 
401
        char *envdef = strdup(arg);
 
402
        if(envdef == NULL){
 
403
          break;
 
404
        }
 
405
        if(not add_environment(getplugin(NULL), envdef, true)){
 
406
          perror("add_environment");
 
407
        }
402
408
      }
403
409
      break;
404
410
    case 'o':                   /* --options-for */
432
438
        if(envdef == NULL){
433
439
          break;
434
440
        }
435
 
        *envdef = '\0';
436
 
        if(not add_environment(getplugin(arg), envdef+1, true)){
 
441
        char *p_name = strndup(arg, (size_t) (envdef-arg));
 
442
        if(p_name == NULL){
 
443
          break;
 
444
        }
 
445
        envdef++;
 
446
        if(not add_environment(getplugin(p_name), envdef, true)){
437
447
          perror("add_environment");
438
448
        }
439
449
      }
457
467
      }
458
468
      break;
459
469
    case 128:                   /* --plugin-dir */
460
 
      free(plugindir);
461
470
      plugindir = strdup(arg);
462
471
      if(plugindir == NULL){
463
472
        perror("strdup");
501
510
    case 128:                   /* --plugin-dir */
502
511
      break;
503
512
    case 129:                   /* --config-file */
504
 
      free(argfile);
505
513
      argfile = strdup(arg);
506
514
      if(argfile == NULL){
507
515
        perror("strdup");
932
940
    }
933
941
    /* OK, now either a process completed, or something can be read
934
942
       from one of them */
935
 
    for(plugin *proc = plugin_list; proc != NULL;){
 
943
    for(plugin *proc = plugin_list; proc != NULL; proc = proc->next){
936
944
      /* Is this process completely done? */
937
945
      if(proc->eof and proc->completed){
938
946
        /* Only accept the plugin output if it exited cleanly */
965
973
            exitstatus = EXIT_FAILURE;
966
974
            goto fallback;
967
975
          }
968
 
          
 
976
          free_plugin(proc);
969
977
          /* We are done modifying process list, so unblock signal */
970
978
          ret = sigprocmask (SIG_UNBLOCK, &sigchld_action.sa_mask,
971
979
                             NULL);
978
986
          if(plugin_list == NULL){
979
987
            break;
980
988
          }
981
 
          
982
 
          plugin *next_plugin = proc->next;
983
 
          free_plugin(proc);
984
 
          proc = next_plugin;
985
989
          continue;
986
990
        }
987
991
        
999
1003
      /* This process has not completed.  Does it have any output? */
1000
1004
      if(proc->eof or not FD_ISSET(proc->fd, &rfds)){
1001
1005
        /* This process had nothing to say at this time */
1002
 
        proc = proc->next;
1003
1006
        continue;
1004
1007
      }
1005
1008
      /* Before reading, make the process' data buffer large enough */
1018
1021
                 BUFFER_SIZE);
1019
1022
      if(ret < 0){
1020
1023
        /* Read error from this process; ignore the error */
1021
 
        proc = proc->next;
1022
1024
        continue;
1023
1025
      }
1024
1026
      if(ret == 0){