/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-07 01:44:44 UTC
  • mfrom: (24.1.93 mandos)
  • Revision ID: teddy@fukt.bsnet.se-20080907014444-cf4ilzndc0tbn8va
Merge & resolve.

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");
507
498
    case 128:                   /* --plugin-dir */
508
499
      break;
509
500
    case 129:                   /* --config-file */
 
501
      free(argfile);
510
502
      argfile = strdup(arg);
511
503
      if(argfile == NULL){
512
504
        perror("strdup");
941
933
    }
942
934
    /* OK, now either a process completed, or something can be read
943
935
       from one of them */
944
 
    for(plugin *proc = plugin_list; proc != NULL; proc = proc->next){
 
936
    for(plugin *proc = plugin_list; proc != NULL;){
945
937
      /* Is this process completely done? */
946
938
      if(proc->eof and proc->completed){
947
939
        /* Only accept the plugin output if it exited cleanly */
974
966
            exitstatus = EXIT_FAILURE;
975
967
            goto fallback;
976
968
          }
977
 
          free_plugin(proc);
 
969
          
978
970
          /* We are done modifying process list, so unblock signal */
979
971
          ret = sigprocmask (SIG_UNBLOCK, &sigchld_action.sa_mask,
980
972
                             NULL);
987
979
          if(plugin_list == NULL){
988
980
            break;
989
981
          }
 
982
          
 
983
          plugin *next_plugin = proc->next;
 
984
          free_plugin(proc);
 
985
          proc = next_plugin;
990
986
          continue;
991
987
        }
992
988
        
1004
1000
      /* This process has not completed.  Does it have any output? */
1005
1001
      if(proc->eof or not FD_ISSET(proc->fd, &rfds)){
1006
1002
        /* This process had nothing to say at this time */
 
1003
        proc = proc->next;
1007
1004
        continue;
1008
1005
      }
1009
1006
      /* Before reading, make the process' data buffer large enough */
1022
1019
                 BUFFER_SIZE);
1023
1020
      if(ret < 0){
1024
1021
        /* Read error from this process; ignore the error */
 
1022
        proc = proc->next;
1025
1023
        continue;
1026
1024
      }
1027
1025
      if(ret == 0){