/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

* plugins.d/password-prompt.c: Use environment variables and prompt
                               text from cryptsetup 1.1.
* plugins.d/password-prompt.xml (ENVIRONMENT): Document change in
                                               environment variables
                                               used.

Show diffs side-by-side

added added

removed removed

Lines of Context:
plugin-runner.c
2
2
/*
3
3
 * Mandos plugin runner - Run Mandos plugins
4
4
 *
5
 
 * Copyright © 2008-2012 Teddy Hogeborn
6
 
 * Copyright © 2008-2012 Björn Påhlsson
 
5
 * Copyright © 2008,2009 Teddy Hogeborn
 
6
 * Copyright © 2008,2009 Björn Påhlsson
7
7
 * 
8
8
 * This program is free software: you can redistribute it and/or
9
9
 * modify it under the terms of the GNU General Public License as
19
19
 * along with this program.  If not, see
20
20
 * <http://www.gnu.org/licenses/>.
21
21
 * 
22
 
 * Contact the authors at <mandos@recompile.se>.
 
22
 * Contact the authors at <mandos@fukt.bsnet.se>.
23
23
 */
24
24
 
25
25
#define _GNU_SOURCE             /* TEMP_FAILURE_RETRY(), getline(),
26
26
                                   asprintf(), O_CLOEXEC */
27
27
#include <stddef.h>             /* size_t, NULL */
28
 
#include <stdlib.h>             /* malloc(), exit(), EXIT_SUCCESS,
29
 
                                   realloc() */
 
28
#include <stdlib.h>             /* malloc(), exit(), EXIT_FAILURE,
 
29
                                   EXIT_SUCCESS, realloc() */
30
30
#include <stdbool.h>            /* bool, true, false */
31
 
#include <stdio.h>              /* fileno(), fprintf(),
 
31
#include <stdio.h>              /* perror, fileno(), fprintf(),
32
32
                                   stderr, STDOUT_FILENO */
33
33
#include <sys/types.h>          /* DIR, fdopendir(), stat(), struct
34
34
                                   stat, waitpid(), WIFEXITED(),
54
54
#include <fcntl.h>              /* fcntl(), F_GETFD, F_SETFD,
55
55
                                   FD_CLOEXEC */
56
56
#include <string.h>             /* strsep, strlen(), asprintf(),
57
 
                                   strsignal(), strcmp(), strncmp() */
 
57
                                   strsignal() */
58
58
#include <errno.h>              /* errno */
59
59
#include <argp.h>               /* struct argp_option, struct
60
60
                                   argp_state, struct argp,
68
68
                                */
69
69
#include <errno.h>              /* errno, EBADF */
70
70
#include <inttypes.h>           /* intmax_t, PRIdMAX, strtoimax() */
71
 
#include <sysexits.h>           /* EX_OSERR, EX_USAGE, EX_IOERR,
72
 
                                   EX_CONFIG, EX_UNAVAILABLE, EX_OK */
73
 
#include <errno.h>              /* errno */
74
 
#include <error.h>              /* error() */
 
71
#include <sysexits.h>           /* EX_OSERR, EX_USAGE */
75
72
 
76
73
#define BUFFER_SIZE 256
77
74
 
79
76
#define AFILE "/conf/conf.d/mandos/plugin-runner.conf"
80
77
 
81
78
const char *argp_program_version = "plugin-runner " VERSION;
82
 
const char *argp_program_bug_address = "<mandos@recompile.se>";
 
79
const char *argp_program_bug_address = "<mandos@fukt.bsnet.se>";
83
80
 
84
81
typedef struct plugin{
85
82
  char *name;                   /* can be NULL or any plugin name */
171
168
}
172
169
 
173
170
/* Helper function for add_argument and add_environment */
174
 
__attribute__((nonnull))
175
171
static bool add_to_char_array(const char *new, char ***array,
176
172
                              int *len){
177
173
  /* Resize the pointed-to array to hold one more pointer */
200
196
}
201
197
 
202
198
/* Add to a plugin's argument vector */
203
 
__attribute__((nonnull(2)))
204
199
static bool add_argument(plugin *p, const char *arg){
205
200
  if(p == NULL){
206
201
    return false;
209
204
}
210
205
 
211
206
/* Add to a plugin's environment */
212
 
__attribute__((nonnull(2)))
213
207
static bool add_environment(plugin *p, const char *def, bool replace){
214
208
  if(p == NULL){
215
209
    return false;
271
265
        /* No child processes */
272
266
        break;
273
267
      }
274
 
      error(0, errno, "waitpid");
 
268
      perror("waitpid");
275
269
    }
276
270
    
277
271
    /* A child exited, find it in process_list */
289
283
}
290
284
 
291
285
/* Prints out a password to stdout */
292
 
__attribute__((nonnull))
293
286
static bool print_out_password(const char *buffer, size_t length){
294
287
  ssize_t ret;
295
288
  for(size_t written = 0; written < length; written += (size_t)ret){
303
296
}
304
297
 
305
298
/* Removes and free a plugin from the plugin list */
306
 
__attribute__((nonnull))
307
299
static void free_plugin(plugin *plugin_node){
308
300
  
309
301
  for(char **arg = plugin_node->argv; *arg != NULL; arg++){
364
356
  sigemptyset(&sigchld_action.sa_mask);
365
357
  ret = sigaddset(&sigchld_action.sa_mask, SIGCHLD);
366
358
  if(ret == -1){
367
 
    error(0, errno, "sigaddset");
368
 
    exitstatus = EX_OSERR;
 
359
    perror("sigaddset");
 
360
    exitstatus = EXIT_FAILURE;
369
361
    goto fallback;
370
362
  }
371
363
  ret = sigaction(SIGCHLD, &sigchld_action, &old_sigchld_action);
372
364
  if(ret == -1){
373
 
    error(0, errno, "sigaction");
374
 
    exitstatus = EX_OSERR;
 
365
    perror("sigaction");
 
366
    exitstatus = EXIT_FAILURE;
375
367
    goto fallback;
376
368
  }
377
369
  
421
413
    { .name = NULL }
422
414
  };
423
415
  
424
 
  __attribute__((nonnull(3)))
425
416
  error_t parse_opt(int key, char *arg, struct argp_state *state){
426
417
    errno = 0;
427
418
    switch(key){
428
419
      char *tmp;
429
 
      intmax_t tmp_id;
 
420
      intmax_t tmpmax;
430
421
    case 'g':                   /* --global-options */
431
422
      {
432
423
        char *plugin_option;
505
496
      /* This is already done by parse_opt_config_file() */
506
497
      break;
507
498
    case 130:                   /* --userid */
508
 
      tmp_id = strtoimax(arg, &tmp, 10);
 
499
      tmpmax = strtoimax(arg, &tmp, 10);
509
500
      if(errno != 0 or tmp == arg or *tmp != '\0'
510
 
         or tmp_id != (uid_t)tmp_id){
 
501
         or tmpmax != (uid_t)tmpmax){
511
502
        argp_error(state, "Bad user ID number: \"%s\", using %"
512
503
                   PRIdMAX, arg, (intmax_t)uid);
513
504
        break;
514
505
      }
515
 
      uid = (uid_t)tmp_id;
 
506
      uid = (uid_t)tmpmax;
516
507
      break;
517
508
    case 131:                   /* --groupid */
518
 
      tmp_id = strtoimax(arg, &tmp, 10);
 
509
      tmpmax = strtoimax(arg, &tmp, 10);
519
510
      if(errno != 0 or tmp == arg or *tmp != '\0'
520
 
         or tmp_id != (gid_t)tmp_id){
 
511
         or tmpmax != (gid_t)tmpmax){
521
512
        argp_error(state, "Bad group ID number: \"%s\", using %"
522
513
                   PRIdMAX, arg, (intmax_t)gid);
523
514
        break;
524
515
      }
525
 
      gid = (gid_t)tmp_id;
 
516
      gid = (gid_t)tmpmax;
526
517
      break;
527
518
    case 132:                   /* --debug */
528
519
      debug = true;
607
598
  case ENOMEM:
608
599
  default:
609
600
    errno = ret;
610
 
    error(0, errno, "argp_parse");
 
601
    perror("argp_parse");
611
602
    exitstatus = EX_OSERR;
612
603
    goto fallback;
613
604
  case EINVAL:
634
625
    custom_argc = 1;
635
626
    custom_argv = malloc(sizeof(char*) * 2);
636
627
    if(custom_argv == NULL){
637
 
      error(0, errno, "malloc");
638
 
      exitstatus = EX_OSERR;
 
628
      perror("malloc");
 
629
      exitstatus = EXIT_FAILURE;
639
630
      goto fallback;
640
631
    }
641
632
    custom_argv[0] = argv[0];
657
648
        }
658
649
        new_arg = strdup(p);
659
650
        if(new_arg == NULL){
660
 
          error(0, errno, "strdup");
661
 
          exitstatus = EX_OSERR;
 
651
          perror("strdup");
 
652
          exitstatus = EXIT_FAILURE;
662
653
          free(org_line);
663
654
          goto fallback;
664
655
        }
667
658
        custom_argv = realloc(custom_argv, sizeof(char *)
668
659
                              * ((unsigned int) custom_argc + 1));
669
660
        if(custom_argv == NULL){
670
 
          error(0, errno, "realloc");
671
 
          exitstatus = EX_OSERR;
 
661
          perror("realloc");
 
662
          exitstatus = EXIT_FAILURE;
672
663
          free(org_line);
673
664
          goto fallback;
674
665
        }
680
671
      ret = fclose(conffp);
681
672
    } while(ret == EOF and errno == EINTR);
682
673
    if(ret == EOF){
683
 
      error(0, errno, "fclose");
684
 
      exitstatus = EX_IOERR;
 
674
      perror("fclose");
 
675
      exitstatus = EXIT_FAILURE;
685
676
      goto fallback;
686
677
    }
687
678
    free(org_line);
689
680
    /* Check for harmful errors and go to fallback. Other errors might
690
681
       not affect opening plugins */
691
682
    if(errno == EMFILE or errno == ENFILE or errno == ENOMEM){
692
 
      error(0, errno, "fopen");
693
 
      exitstatus = EX_OSERR;
 
683
      perror("fopen");
 
684
      exitstatus = EXIT_FAILURE;
694
685
      goto fallback;
695
686
    }
696
687
  }
706
697
    case ENOMEM:
707
698
    default:
708
699
      errno = ret;
709
 
      error(0, errno, "argp_parse");
 
700
      perror("argp_parse");
710
701
      exitstatus = EX_OSERR;
711
702
      goto fallback;
712
703
    case EINVAL:
726
717
  case ENOMEM:
727
718
  default:
728
719
    errno = ret;
729
 
    error(0, errno, "argp_parse");
 
720
    perror("argp_parse");
730
721
    exitstatus = EX_OSERR;
731
722
    goto fallback;
732
723
  case EINVAL:
748
739
    }
749
740
  }
750
741
  
751
 
  if(getuid() == 0){
752
 
    /* Work around Debian bug #633582:
753
 
       <http://bugs.debian.org/633582> */
754
 
    int plugindir_fd = open(/* plugindir or */ PDIR, O_RDONLY);
755
 
    if(plugindir_fd == -1){
756
 
      error(0, errno, "open");
757
 
    } else {
758
 
      ret = (int)TEMP_FAILURE_RETRY(fstat(plugindir_fd, &st));
759
 
      if(ret == -1){
760
 
        error(0, errno, "fstat");
761
 
      } else {
762
 
        if(S_ISDIR(st.st_mode) and st.st_uid == 0 and st.st_gid == 0){
763
 
          ret = fchown(plugindir_fd, uid, gid);
764
 
          if(ret == -1){
765
 
            error(0, errno, "fchown");
766
 
          }
767
 
        }
768
 
      }
769
 
      TEMP_FAILURE_RETRY(close(plugindir_fd));
770
 
    }
771
 
  }
772
 
  
773
 
  /* Lower permissions */
 
742
  /* Strip permissions down to nobody */
774
743
  setgid(gid);
775
744
  if(ret == -1){
776
 
    error(0, errno, "setgid");
 
745
    perror("setgid");
777
746
  }
778
747
  ret = setuid(uid);
779
748
  if(ret == -1){
780
 
    error(0, errno, "setuid");
 
749
    perror("setuid");
781
750
  }
782
751
  
783
752
  /* Open plugin directory with close_on_exec flag */
801
770
                    );
802
771
    }
803
772
    if(dir_fd == -1){
804
 
      error(0, errno, "Could not open plugin dir");
805
 
      exitstatus = EX_UNAVAILABLE;
 
773
      perror("Could not open plugin dir");
 
774
      exitstatus = EXIT_FAILURE;
806
775
      goto fallback;
807
776
    }
808
777
    
810
779
  /* Set the FD_CLOEXEC flag on the directory */
811
780
    ret = set_cloexec_flag(dir_fd);
812
781
    if(ret < 0){
813
 
      error(0, errno, "set_cloexec_flag");
 
782
      perror("set_cloexec_flag");
814
783
      TEMP_FAILURE_RETRY(close(dir_fd));
815
 
      exitstatus = EX_OSERR;
 
784
      exitstatus = EXIT_FAILURE;
816
785
      goto fallback;
817
786
    }
818
787
#endif  /* O_CLOEXEC */
819
788
    
820
789
    dir = fdopendir(dir_fd);
821
790
    if(dir == NULL){
822
 
      error(0, errno, "Could not open plugin dir");
 
791
      perror("Could not open plugin dir");
823
792
      TEMP_FAILURE_RETRY(close(dir_fd));
824
 
      exitstatus = EX_OSERR;
 
793
      exitstatus = EXIT_FAILURE;
825
794
      goto fallback;
826
795
    }
827
796
  }
837
806
    /* All directory entries have been processed */
838
807
    if(dirst == NULL){
839
808
      if(errno == EBADF){
840
 
        error(0, errno, "readdir");
841
 
        exitstatus = EX_IOERR;
 
809
        perror("readdir");
 
810
        exitstatus = EXIT_FAILURE;
842
811
        goto fallback;
843
812
      }
844
813
      break;
874
843
      for(const char **suf = bad_suffixes; *suf != NULL; suf++){
875
844
        size_t suf_len = strlen(*suf);
876
845
        if((d_name_len >= suf_len)
877
 
           and (strcmp((dirst->d_name) + d_name_len-suf_len, *suf)
 
846
           and (strcmp((dirst->d_name)+d_name_len-suf_len, *suf)
878
847
                == 0)){
879
848
          if(debug){
880
849
            fprintf(stderr, "Ignoring plugin dir entry \"%s\""
900
869
                                             dirst->d_name));
901
870
    }
902
871
    if(ret < 0){
903
 
      error(0, errno, "asprintf");
 
872
      perror("asprintf");
904
873
      continue;
905
874
    }
906
875
    
907
876
    ret = (int)TEMP_FAILURE_RETRY(stat(filename, &st));
908
877
    if(ret == -1){
909
 
      error(0, errno, "stat");
 
878
      perror("stat");
910
879
      free(filename);
911
880
      continue;
912
881
    }
924
893
    
925
894
    plugin *p = getplugin(dirst->d_name);
926
895
    if(p == NULL){
927
 
      error(0, errno, "getplugin");
 
896
      perror("getplugin");
928
897
      free(filename);
929
898
      continue;
930
899
    }
942
911
      if(g != NULL){
943
912
        for(char **a = g->argv + 1; *a != NULL; a++){
944
913
          if(not add_argument(p, *a)){
945
 
            error(0, errno, "add_argument");
 
914
            perror("add_argument");
946
915
          }
947
916
        }
948
917
        /* Add global environment variables */
949
918
        for(char **e = g->environ; *e != NULL; e++){
950
919
          if(not add_environment(p, *e, false)){
951
 
            error(0, errno, "add_environment");
 
920
            perror("add_environment");
952
921
          }
953
922
        }
954
923
      }
959
928
    if(p->environ[0] != NULL){
960
929
      for(char **e = environ; *e != NULL; e++){
961
930
        if(not add_environment(p, *e, false)){
962
 
          error(0, errno, "add_environment");
 
931
          perror("add_environment");
963
932
        }
964
933
      }
965
934
    }
967
936
    int pipefd[2];
968
937
    ret = (int)TEMP_FAILURE_RETRY(pipe(pipefd));
969
938
    if(ret == -1){
970
 
      error(0, errno, "pipe");
971
 
      exitstatus = EX_OSERR;
 
939
      perror("pipe");
 
940
      exitstatus = EXIT_FAILURE;
972
941
      goto fallback;
973
942
    }
974
943
    /* Ask OS to automatic close the pipe on exec */
975
944
    ret = set_cloexec_flag(pipefd[0]);
976
945
    if(ret < 0){
977
 
      error(0, errno, "set_cloexec_flag");
978
 
      exitstatus = EX_OSERR;
 
946
      perror("set_cloexec_flag");
 
947
      exitstatus = EXIT_FAILURE;
979
948
      goto fallback;
980
949
    }
981
950
    ret = set_cloexec_flag(pipefd[1]);
982
951
    if(ret < 0){
983
 
      error(0, errno, "set_cloexec_flag");
984
 
      exitstatus = EX_OSERR;
 
952
      perror("set_cloexec_flag");
 
953
      exitstatus = EXIT_FAILURE;
985
954
      goto fallback;
986
955
    }
987
956
    /* Block SIGCHLD until process is safely in process list */
989
958
                                              &sigchld_action.sa_mask,
990
959
                                              NULL));
991
960
    if(ret < 0){
992
 
      error(0, errno, "sigprocmask");
993
 
      exitstatus = EX_OSERR;
 
961
      perror("sigprocmask");
 
962
      exitstatus = EXIT_FAILURE;
994
963
      goto fallback;
995
964
    }
996
965
    /* Starting a new process to be watched */
999
968
      pid = fork();
1000
969
    } while(pid == -1 and errno == EINTR);
1001
970
    if(pid == -1){
1002
 
      error(0, errno, "fork");
1003
 
      exitstatus = EX_OSERR;
 
971
      perror("fork");
 
972
      exitstatus = EXIT_FAILURE;
1004
973
      goto fallback;
1005
974
    }
1006
975
    if(pid == 0){
1007
976
      /* this is the child process */
1008
977
      ret = sigaction(SIGCHLD, &old_sigchld_action, NULL);
1009
978
      if(ret < 0){
1010
 
        error(0, errno, "sigaction");
1011
 
        _exit(EX_OSERR);
 
979
        perror("sigaction");
 
980
        _exit(EXIT_FAILURE);
1012
981
      }
1013
982
      ret = sigprocmask(SIG_UNBLOCK, &sigchld_action.sa_mask, NULL);
1014
983
      if(ret < 0){
1015
 
        error(0, errno, "sigprocmask");
1016
 
        _exit(EX_OSERR);
 
984
        perror("sigprocmask");
 
985
        _exit(EXIT_FAILURE);
1017
986
      }
1018
987
      
1019
988
      ret = dup2(pipefd[1], STDOUT_FILENO); /* replace our stdout */
1020
989
      if(ret == -1){
1021
 
        error(0, errno, "dup2");
1022
 
        _exit(EX_OSERR);
 
990
        perror("dup2");
 
991
        _exit(EXIT_FAILURE);
1023
992
      }
1024
993
      
1025
994
      if(dirfd(dir) < 0){
1029
998
      }
1030
999
      if(p->environ[0] == NULL){
1031
1000
        if(execv(filename, p->argv) < 0){
1032
 
          error(0, errno, "execv for %s", filename);
1033
 
          _exit(EX_OSERR);
 
1001
          perror("execv");
 
1002
          _exit(EXIT_FAILURE);
1034
1003
        }
1035
1004
      } else {
1036
1005
        if(execve(filename, p->argv, p->environ) < 0){
1037
 
          error(0, errno, "execve for %s", filename);
1038
 
          _exit(EX_OSERR);
 
1006
          perror("execve");
 
1007
          _exit(EXIT_FAILURE);
1039
1008
        }
1040
1009
      }
1041
1010
      /* no return */
1046
1015
    free(filename);
1047
1016
    plugin *new_plugin = getplugin(dirst->d_name);
1048
1017
    if(new_plugin == NULL){
1049
 
      error(0, errno, "getplugin");
 
1018
      perror("getplugin");
1050
1019
      ret = (int)(TEMP_FAILURE_RETRY
1051
1020
                  (sigprocmask(SIG_UNBLOCK, &sigchld_action.sa_mask,
1052
1021
                               NULL)));
1053
1022
      if(ret < 0){
1054
 
        error(0, errno, "sigprocmask");
 
1023
        perror("sigprocmask");
1055
1024
      }
1056
 
      exitstatus = EX_OSERR;
 
1025
      exitstatus = EXIT_FAILURE;
1057
1026
      goto fallback;
1058
1027
    }
1059
1028
    
1066
1035
                                              &sigchld_action.sa_mask,
1067
1036
                                              NULL));
1068
1037
    if(ret < 0){
1069
 
      error(0, errno, "sigprocmask");
1070
 
      exitstatus = EX_OSERR;
 
1038
      perror("sigprocmask");
 
1039
      exitstatus = EXIT_FAILURE;
1071
1040
      goto fallback;
1072
1041
    }
1073
1042
    
1099
1068
    fd_set rfds = rfds_all;
1100
1069
    int select_ret = select(maxfd+1, &rfds, NULL, NULL, NULL);
1101
1070
    if(select_ret == -1 and errno != EINTR){
1102
 
      error(0, errno, "select");
1103
 
      exitstatus = EX_OSERR;
 
1071
      perror("select");
 
1072
      exitstatus = EXIT_FAILURE;
1104
1073
      goto fallback;
1105
1074
    }
1106
1075
    /* OK, now either a process completed, or something can be read
1141
1110
                                         &sigchld_action.sa_mask,
1142
1111
                                         NULL));
1143
1112
          if(ret < 0){
1144
 
            error(0, errno, "sigprocmask");
1145
 
            exitstatus = EX_OSERR;
 
1113
            perror("sigprocmask");
 
1114
            exitstatus = EXIT_FAILURE;
1146
1115
            goto fallback;
1147
1116
          }
1148
1117
          
1155
1124
                      (sigprocmask(SIG_UNBLOCK,
1156
1125
                                   &sigchld_action.sa_mask, NULL)));
1157
1126
          if(ret < 0){
1158
 
            error(0, errno, "sigprocmask");
1159
 
            exitstatus = EX_OSERR;
 
1127
            perror("sigprocmask");
 
1128
            exitstatus = EXIT_FAILURE;
1160
1129
            goto fallback;
1161
1130
          }
1162
1131
          
1172
1141
        bool bret = print_out_password(proc->buffer,
1173
1142
                                       proc->buffer_length);
1174
1143
        if(not bret){
1175
 
          error(0, errno, "print_out_password");
1176
 
          exitstatus = EX_IOERR;
 
1144
          perror("print_out_password");
 
1145
          exitstatus = EXIT_FAILURE;
1177
1146
        }
1178
1147
        goto fallback;
1179
1148
      }
1191
1160
        proc->buffer = realloc(proc->buffer, proc->buffer_size
1192
1161
                               + (size_t) BUFFER_SIZE);
1193
1162
        if(proc->buffer == NULL){
1194
 
          error(0, errno, "malloc");
1195
 
          exitstatus = EX_OSERR;
 
1163
          perror("malloc");
 
1164
          exitstatus = EXIT_FAILURE;
1196
1165
          goto fallback;
1197
1166
        }
1198
1167
        proc->buffer_size += BUFFER_SIZE;
1219
1188
  
1220
1189
 fallback:
1221
1190
  
1222
 
  if(plugin_list == NULL or (exitstatus != EXIT_SUCCESS
1223
 
                             and exitstatus != EX_OK)){
 
1191
  if(plugin_list == NULL or exitstatus != EXIT_SUCCESS){
1224
1192
    /* Fallback if all plugins failed, none are found or an error
1225
1193
       occured */
1226
1194
    bool bret;
1234
1202
    }
1235
1203
    bret = print_out_password(passwordbuffer, len);
1236
1204
    if(not bret){
1237
 
      error(0, errno, "print_out_password");
1238
 
      exitstatus = EX_IOERR;
 
1205
      perror("print_out_password");
 
1206
      exitstatus = EXIT_FAILURE;
1239
1207
    }
1240
1208
  }
1241
1209
  
1242
1210
  /* Restore old signal handler */
1243
1211
  ret = sigaction(SIGCHLD, &old_sigchld_action, NULL);
1244
1212
  if(ret == -1){
1245
 
    error(0, errno, "sigaction");
1246
 
    exitstatus = EX_OSERR;
 
1213
    perror("sigaction");
 
1214
    exitstatus = EXIT_FAILURE;
1247
1215
  }
1248
1216
  
1249
1217
  if(custom_argv != NULL){
1264
1232
      ret = kill(p->pid, SIGTERM);
1265
1233
      if(ret == -1 and errno != ESRCH){
1266
1234
        /* Set-uid proccesses might not get closed */
1267
 
        error(0, errno, "kill");
 
1235
        perror("kill");
1268
1236
      }
1269
1237
    }
1270
1238
  }
1274
1242
    ret = wait(NULL);
1275
1243
  } while(ret >= 0);
1276
1244
  if(errno != ECHILD){
1277
 
    error(0, errno, "wait");
 
1245
    perror("wait");
1278
1246
  }
1279
1247
  
1280
1248
  free_plugin_list();