/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

* plugin-runner.c (handle_sigchld): Loop until all exited children
                                    have been wait()ed for.

Show diffs side-by-side

added added

removed removed

Lines of Context:
2
2
/*
3
3
 * Mandos plugin runner - Run Mandos plugins
4
4
 *
5
 
 * Copyright © 2008,2009 Teddy Hogeborn
6
 
 * Copyright © 2008,2009 Björn Påhlsson
 
5
 * Copyright © 2007-2008 Teddy Hogeborn & Björn Påhlsson
7
6
 * 
8
7
 * This program is free software: you can redistribute it and/or
9
8
 * modify it under the terms of the GNU General Public License as
28
27
#include <stdlib.h>             /* malloc(), exit(), EXIT_FAILURE,
29
28
                                   EXIT_SUCCESS, realloc() */
30
29
#include <stdbool.h>            /* bool, true, false */
31
 
#include <stdio.h>              /* perror, fileno(), fprintf(),
32
 
                                   stderr, STDOUT_FILENO */
 
30
#include <stdio.h>              /* perror, popen(), fileno(),
 
31
                                   fprintf(), stderr, STDOUT_FILENO */
33
32
#include <sys/types.h>          /* DIR, opendir(), stat(), struct
34
33
                                   stat, waitpid(), WIFEXITED(),
35
34
                                   WEXITSTATUS(), wait(), pid_t,
47
46
                                   fcntl(), setuid(), setgid(),
48
47
                                   F_GETFD, F_SETFD, FD_CLOEXEC,
49
48
                                   access(), pipe(), fork(), close()
50
 
                                   dup2(), STDOUT_FILENO, _exit(),
 
49
                                   dup2, STDOUT_FILENO, _exit(),
51
50
                                   execv(), write(), read(),
52
51
                                   close() */
53
52
#include <fcntl.h>              /* fcntl(), F_GETFD, F_SETFD,
57
56
#include <argp.h>               /* struct argp_option, struct
58
57
                                   argp_state, struct argp,
59
58
                                   argp_parse(), ARGP_ERR_UNKNOWN,
60
 
                                   ARGP_KEY_END, ARGP_KEY_ARG,
61
 
                                   error_t */
 
59
                                   ARGP_KEY_END, ARGP_KEY_ARG, error_t */
62
60
#include <signal.h>             /* struct sigaction, sigemptyset(),
63
61
                                   sigaddset(), sigaction(),
64
62
                                   sigprocmask(), SIG_BLOCK, SIGCHLD,
65
 
                                   SIG_UNBLOCK, kill(), sig_atomic_t
66
 
                                */
 
63
                                   SIG_UNBLOCK, kill() */
67
64
#include <errno.h>              /* errno, EBADF */
68
 
#include <inttypes.h>           /* intmax_t, PRIdMAX, strtoimax() */
69
65
 
70
66
#define BUFFER_SIZE 256
71
 
 
72
 
#define PDIR "/lib/mandos/plugins.d"
73
 
#define AFILE "/conf/conf.d/mandos/plugin-runner.conf"
74
 
 
75
 
const char *argp_program_version = "plugin-runner " VERSION;
 
67
#define ARGFILE "/conf/conf.d/mandos/plugin-runner.conf"
 
68
 
 
69
const char *argp_program_version = "plugin-runner 1.0";
76
70
const char *argp_program_bug_address = "<mandos@fukt.bsnet.se>";
77
71
 
 
72
struct process;
 
73
 
 
74
typedef struct process{
 
75
  pid_t pid;
 
76
  int fd;
 
77
  char *buffer;
 
78
  size_t buffer_size;
 
79
  size_t buffer_length;
 
80
  bool eof;
 
81
  bool completed;
 
82
  int status;
 
83
  struct process *next;
 
84
} process;
 
85
 
78
86
typedef struct plugin{
79
87
  char *name;                   /* can be NULL or any plugin name */
80
88
  char **argv;
82
90
  char **environ;
83
91
  int envc;
84
92
  bool disabled;
85
 
  
86
 
  /* Variables used for running processes*/
87
 
  pid_t pid;
88
 
  int fd;
89
 
  char *buffer;
90
 
  size_t buffer_size;
91
 
  size_t buffer_length;
92
 
  bool eof;
93
 
  volatile sig_atomic_t completed;
94
 
  int status;
95
93
  struct plugin *next;
96
94
} plugin;
97
95
 
98
 
static plugin *plugin_list = NULL;
99
 
 
100
 
/* Gets an existing plugin based on name,
101
 
   or if none is found, creates a new one */
102
 
static plugin *getplugin(char *name){
103
 
  /* Check for exiting plugin with that name */
104
 
  for(plugin *p = plugin_list; p != NULL; p = p->next){
105
 
    if((p->name == name)
106
 
       or (p->name and name and (strcmp(p->name, name) == 0))){
 
96
static plugin *getplugin(char *name, plugin **plugin_list){
 
97
  for (plugin *p = *plugin_list; p != NULL; p = p->next){
 
98
    if ((p->name == name)
 
99
        or (p->name and name and (strcmp(p->name, name) == 0))){
107
100
      return p;
108
101
    }
109
102
  }
110
103
  /* Create a new plugin */
111
104
  plugin *new_plugin = malloc(sizeof(plugin));
112
 
  if(new_plugin == NULL){
 
105
  if (new_plugin == NULL){
113
106
    return NULL;
114
107
  }
115
108
  char *copy_name = NULL;
116
109
  if(name != NULL){
117
110
    copy_name = strdup(name);
118
111
    if(copy_name == NULL){
119
 
      free(new_plugin);
120
112
      return NULL;
121
113
    }
122
114
  }
123
115
  
124
 
  *new_plugin = (plugin){ .name = copy_name,
125
 
                          .argc = 1,
126
 
                          .disabled = false,
127
 
                          .next = plugin_list };
 
116
  *new_plugin = (plugin) { .name = copy_name,
 
117
                           .argc = 1,
 
118
                           .envc = 0,
 
119
                           .disabled = false,
 
120
                           .next = *plugin_list };
128
121
  
129
122
  new_plugin->argv = malloc(sizeof(char *) * 2);
130
 
  if(new_plugin->argv == NULL){
 
123
  if (new_plugin->argv == NULL){
131
124
    free(copy_name);
132
125
    free(new_plugin);
133
126
    return NULL;
134
127
  }
135
128
  new_plugin->argv[0] = copy_name;
136
129
  new_plugin->argv[1] = NULL;
137
 
  
 
130
 
138
131
  new_plugin->environ = malloc(sizeof(char *));
139
132
  if(new_plugin->environ == NULL){
140
133
    free(copy_name);
143
136
    return NULL;
144
137
  }
145
138
  new_plugin->environ[0] = NULL;
146
 
  
147
139
  /* Append the new plugin to the list */
148
 
  plugin_list = new_plugin;
 
140
  *plugin_list = new_plugin;
149
141
  return new_plugin;
150
142
}
151
143
 
181
173
}
182
174
 
183
175
/* Add to a plugin's environment */
184
 
static bool add_environment(plugin *p, const char *def, bool replace){
 
176
static bool add_environment(plugin *p, const char *def){
185
177
  if(p == NULL){
186
178
    return false;
187
179
  }
188
 
  /* namelen = length of name of environment variable */
189
 
  size_t namelen = (size_t)(strchrnul(def, '=') - def);
190
 
  /* Search for this environment variable */
191
 
  for(char **e = p->environ; *e != NULL; e++){
192
 
    if(strncmp(*e, def, namelen + 1) == 0){
193
 
      /* It already exists */
194
 
      if(replace){
195
 
        char *new = realloc(*e, strlen(def) + 1);
196
 
        if(new == NULL){
197
 
          return false;
198
 
        }
199
 
        *e = new;
200
 
        strcpy(*e, def);
201
 
      }
202
 
      return true;
203
 
    }
204
 
  }
205
180
  return add_to_char_array(def, &(p->environ), &(p->envc));
206
181
}
207
182
 
 
183
 
208
184
/*
209
185
 * Based on the example in the GNU LibC manual chapter 13.13 "File
210
186
 * Descriptor Flags".
211
187
 * *Note File Descriptor Flags:(libc)Descriptor Flags.
212
188
 */
213
 
static int set_cloexec_flag(int fd){
 
189
static int set_cloexec_flag(int fd)
 
190
{
214
191
  int ret = fcntl(fd, F_GETFD, 0);
215
192
  /* If reading the flags failed, return error indication now. */
216
193
  if(ret < 0){
220
197
  return fcntl(fd, F_SETFD, ret | FD_CLOEXEC);
221
198
}
222
199
 
 
200
process *process_list = NULL;
223
201
 
224
 
/* Mark processes as completed when they exit, and save their exit
 
202
/* Mark processes as completed when it exits, and save its exit
225
203
   status. */
226
 
static void handle_sigchld(__attribute__((unused)) int sig){
227
 
  int old_errno = errno;
 
204
void handle_sigchld(__attribute__((unused)) int sig){
 
205
  process *proc = process_list;
228
206
  while(true){
229
 
    plugin *proc = plugin_list;
230
207
    int status;
231
208
    pid_t pid = waitpid(-1, &status, WNOHANG);
232
209
    if(pid == 0){
233
 
      /* Only still running child processes */
234
210
      break;
235
211
    }
236
212
    if(pid == -1){
237
 
      if(errno == ECHILD){
238
 
        /* No child processes */
239
 
        break;
 
213
      if (errno != ECHILD){
 
214
        perror("waitpid");
240
215
      }
241
 
      perror("waitpid");
 
216
      return;
242
217
    }
243
 
    
244
 
    /* A child exited, find it in process_list */
 
218
 
245
219
    while(proc != NULL and proc->pid != pid){
246
220
      proc = proc->next;
247
221
    }
250
224
      continue;
251
225
    }
252
226
    proc->status = status;
253
 
    proc->completed = 1;
 
227
    proc->completed = true;
254
228
  }
255
 
  errno = old_errno;
256
229
}
257
230
 
258
 
/* Prints out a password to stdout */
259
 
static bool print_out_password(const char *buffer, size_t length){
 
231
bool print_out_password(const char *buffer, size_t length){
260
232
  ssize_t ret;
 
233
  if(length>0 and buffer[length-1] == '\n'){
 
234
    length--;
 
235
  }
261
236
  for(size_t written = 0; written < length; written += (size_t)ret){
262
237
    ret = TEMP_FAILURE_RETRY(write(STDOUT_FILENO, buffer + written,
263
238
                                   length - written));
268
243
  return true;
269
244
}
270
245
 
271
 
/* Removes and free a plugin from the plugin list */
272
 
static void free_plugin(plugin *plugin_node){
273
 
  
274
 
  for(char **arg = plugin_node->argv; *arg != NULL; arg++){
275
 
    free(*arg);
276
 
  }
277
 
  free(plugin_node->argv);
278
 
  for(char **env = plugin_node->environ; *env != NULL; env++){
279
 
    free(*env);
280
 
  }
281
 
  free(plugin_node->environ);
282
 
  free(plugin_node->buffer);
283
 
  
284
 
  /* Removes the plugin from the singly-linked list */
285
 
  if(plugin_node == plugin_list){
286
 
    /* First one - simple */
287
 
    plugin_list = plugin_list->next;
288
 
  } else {
289
 
    /* Second one or later */
290
 
    for(plugin *p = plugin_list; p != NULL; p = p->next){
291
 
      if(p->next == plugin_node){
292
 
        p->next = plugin_node->next;
293
 
        break;
294
 
      }
 
246
char **add_to_argv(char **argv, int *argc, char *arg){
 
247
  if (argv == NULL){
 
248
    *argc = 1;
 
249
    argv = malloc(sizeof(char*) * 2);
 
250
    if(argv == NULL){
 
251
      return NULL;
295
252
    }
296
 
  }
297
 
  
298
 
  free(plugin_node);
 
253
    argv[0] = NULL;     /* Will be set to argv[0] in main before parsing */
 
254
    argv[1] = NULL;
 
255
  }
 
256
  *argc += 1;
 
257
  argv = realloc(argv, sizeof(char *)
 
258
                  * ((unsigned int) *argc + 1));
 
259
  if(argv == NULL){
 
260
    return NULL;
 
261
  }
 
262
  argv[*argc-1] = arg;
 
263
  argv[*argc] = NULL;
 
264
  return argv;
299
265
}
300
266
 
301
 
static void free_plugin_list(void){
302
 
  while(plugin_list != NULL){
303
 
    free_plugin(plugin_list);
 
267
static void free_plugin_list(plugin *plugin_list){
 
268
  for(plugin *next; plugin_list != NULL; plugin_list = next){
 
269
    next = plugin_list->next;
 
270
    for(char **arg = plugin_list->argv; *arg != NULL; arg++){
 
271
      free(*arg);
 
272
    }
 
273
    free(plugin_list->argv);
 
274
    for(char **env = plugin_list->environ; *env != NULL; env++){
 
275
      free(*env);
 
276
    }
 
277
    free(plugin_list->environ);
 
278
    free(plugin_list);
304
279
  }
305
280
}
306
281
 
307
282
int main(int argc, char *argv[]){
308
 
  char *plugindir = NULL;
309
 
  char *argfile = NULL;
 
283
  const char *plugindir = "/lib/mandos/plugins.d";
 
284
  const char *argfile = ARGFILE;
310
285
  FILE *conffp;
311
286
  size_t d_name_len;
312
287
  DIR *dir = NULL;
314
289
  struct stat st;
315
290
  fd_set rfds_all;
316
291
  int ret, maxfd = 0;
317
 
  ssize_t sret;
318
 
  intmax_t tmpmax;
319
292
  uid_t uid = 65534;
320
293
  gid_t gid = 65534;
321
294
  bool debug = false;
346
319
    { .name = "global-options", .key = 'g',
347
320
      .arg = "OPTION[,OPTION[,...]]",
348
321
      .doc = "Options passed to all plugins" },
349
 
    { .name = "global-env", .key = 'G',
 
322
    { .name = "global-envs", .key = 'e',
350
323
      .arg = "VAR=value",
351
324
      .doc = "Environment variable passed to all plugins" },
352
325
    { .name = "options-for", .key = 'o',
353
326
      .arg = "PLUGIN:OPTION[,OPTION[,...]]",
354
327
      .doc = "Options passed only to specified plugin" },
355
 
    { .name = "env-for", .key = 'E',
 
328
    { .name = "envs-for", .key = 'f',
356
329
      .arg = "PLUGIN:ENV=value",
357
330
      .doc = "Environment variable passed to specified plugin" },
358
331
    { .name = "disable", .key = 'd',
359
332
      .arg = "PLUGIN",
360
333
      .doc = "Disable a specific plugin", .group = 1 },
361
 
    { .name = "enable", .key = 'e',
362
 
      .arg = "PLUGIN",
363
 
      .doc = "Enable a specific plugin", .group = 1 },
364
334
    { .name = "plugin-dir", .key = 128,
365
335
      .arg = "DIRECTORY",
366
336
      .doc = "Specify a different plugin directory", .group = 2 },
367
 
    { .name = "config-file", .key = 129,
368
 
      .arg = "FILE",
369
 
      .doc = "Specify a different configuration file", .group = 2 },
370
 
    { .name = "userid", .key = 130,
371
 
      .arg = "ID", .flags = 0,
372
 
      .doc = "User ID the plugins will run as", .group = 3 },
373
 
    { .name = "groupid", .key = 131,
374
 
      .arg = "ID", .flags = 0,
375
 
      .doc = "Group ID the plugins will run as", .group = 3 },
376
 
    { .name = "debug", .key = 132,
377
 
      .doc = "Debug mode", .group = 4 },
 
337
    { .name = "userid", .key = 129,
 
338
      .arg = "ID", .flags = 0,
 
339
      .doc = "User ID the plugins will run as", .group = 2 },
 
340
    { .name = "groupid", .key = 130,
 
341
      .arg = "ID", .flags = 0,
 
342
      .doc = "Group ID the plugins will run as", .group = 2 },
 
343
    { .name = "debug", .key = 131,
 
344
      .doc = "Debug mode", .group = 3 },
378
345
    { .name = NULL }
379
346
  };
380
347
  
381
 
  error_t parse_opt(int key, char *arg, __attribute__((unused))
382
 
                    struct argp_state *state){
383
 
    char *tmp;
384
 
    switch(key){
385
 
    case 'g':                   /* --global-options */
386
 
      if(arg != NULL){
387
 
        char *plugin_option;
388
 
        while((plugin_option = strsep(&arg, ",")) != NULL){
389
 
          if(plugin_option[0] == '\0'){
 
348
  error_t parse_opt (int key, char *arg, struct argp_state *state) {
 
349
    /* Get the INPUT argument from `argp_parse', which we know is a
 
350
       pointer to our plugin list pointer. */
 
351
    plugin **plugins = state->input;
 
352
    switch (key) {
 
353
    case 'g':
 
354
      if (arg != NULL){
 
355
        char *p;
 
356
        while((p = strsep(&arg, ",")) != NULL){
 
357
          if(p[0] == '\0'){
390
358
            continue;
391
359
          }
392
 
          if(not add_argument(getplugin(NULL), plugin_option)){
 
360
          if(not add_argument(getplugin(NULL, plugins), p)){
393
361
            perror("add_argument");
394
362
            return ARGP_ERR_UNKNOWN;
395
363
          }
396
364
        }
397
365
      }
398
366
      break;
399
 
    case 'G':                   /* --global-env */
 
367
    case 'e':
400
368
      if(arg == NULL){
401
369
        break;
402
370
      }
403
 
      if(not add_environment(getplugin(NULL), arg, true)){
404
 
        perror("add_environment");
 
371
      {
 
372
        char *envdef = strdup(arg);
 
373
        if(envdef == NULL){
 
374
          break;
 
375
        }
 
376
        if(not add_environment(getplugin(NULL, plugins), envdef)){
 
377
          perror("add_environment");
 
378
        }
405
379
      }
406
380
      break;
407
 
    case 'o':                   /* --options-for */
408
 
      if(arg != NULL){
409
 
        char *plugin_name = strsep(&arg, ":");
410
 
        if(plugin_name[0] == '\0'){
411
 
          break;
412
 
        }
413
 
        char *plugin_option;
414
 
        while((plugin_option = strsep(&arg, ",")) != NULL){
415
 
          if(not add_argument(getplugin(plugin_name), plugin_option)){
416
 
            perror("add_argument");
417
 
            return ARGP_ERR_UNKNOWN;
 
381
    case 'o':
 
382
      if (arg != NULL){
 
383
        char *p_name = strsep(&arg, ":");
 
384
        if(p_name[0] == '\0'){
 
385
          break;
 
386
        }
 
387
        char *opt = strsep(&arg, ":");
 
388
        if(opt[0] == '\0'){
 
389
          break;
 
390
        }
 
391
        if(opt != NULL){
 
392
          char *p;
 
393
          while((p = strsep(&opt, ",")) != NULL){
 
394
            if(p[0] == '\0'){
 
395
              continue;
 
396
            }
 
397
            if(not add_argument(getplugin(p_name, plugins), p)){
 
398
              perror("add_argument");
 
399
              return ARGP_ERR_UNKNOWN;
 
400
            }
418
401
          }
419
402
        }
420
403
      }
421
404
      break;
422
 
    case 'E':                   /* --env-for */
 
405
    case 'f':
423
406
      if(arg == NULL){
424
407
        break;
425
408
      }
428
411
        if(envdef == NULL){
429
412
          break;
430
413
        }
431
 
        *envdef = '\0';
432
 
        if(not add_environment(getplugin(arg), envdef+1, true)){
 
414
        char *p_name = strndup(arg, (size_t) (envdef-arg));
 
415
        if(p_name == NULL){
 
416
          break;
 
417
        }
 
418
        envdef++;
 
419
        if(not add_environment(getplugin(p_name, plugins), envdef)){
433
420
          perror("add_environment");
434
421
        }
435
422
      }
436
423
      break;
437
 
    case 'd':                   /* --disable */
438
 
      if(arg != NULL){
439
 
        plugin *p = getplugin(arg);
 
424
    case 'd':
 
425
      if (arg != NULL){
 
426
        plugin *p = getplugin(arg, plugins);
440
427
        if(p == NULL){
441
428
          return ARGP_ERR_UNKNOWN;
442
429
        }
443
430
        p->disabled = true;
444
431
      }
445
432
      break;
446
 
    case 'e':                   /* --enable */
447
 
      if(arg != NULL){
448
 
        plugin *p = getplugin(arg);
449
 
        if(p == NULL){
450
 
          return ARGP_ERR_UNKNOWN;
451
 
        }
452
 
        p->disabled = false;
453
 
      }
454
 
      break;
455
 
    case 128:                   /* --plugin-dir */
456
 
      free(plugindir);
457
 
      plugindir = strdup(arg);
458
 
      if(plugindir == NULL){
459
 
        perror("strdup");
460
 
      }      
461
 
      break;
462
 
    case 129:                   /* --config-file */
463
 
      /* This is already done by parse_opt_config_file() */
464
 
      break;
465
 
    case 130:                   /* --userid */
466
 
      errno = 0;
467
 
      tmpmax = strtoimax(arg, &tmp, 10);
468
 
      if(errno != 0 or tmp == arg or *tmp != '\0'
469
 
         or tmpmax != (uid_t)tmpmax){
470
 
        fprintf(stderr, "Bad user ID number: \"%s\", using %"
471
 
                PRIdMAX "\n", arg, (intmax_t)uid);
472
 
      } else {
473
 
        uid = (uid_t)tmpmax;
474
 
      }
475
 
      break;
476
 
    case 131:                   /* --groupid */
477
 
      errno = 0;
478
 
      tmpmax = strtoimax(arg, &tmp, 10);
479
 
      if(errno != 0 or tmp == arg or *tmp != '\0'
480
 
         or tmpmax != (gid_t)tmpmax){
481
 
        fprintf(stderr, "Bad group ID number: \"%s\", using %"
482
 
                PRIdMAX "\n", arg, (intmax_t)gid);
483
 
      } else {
484
 
        gid = (gid_t)tmpmax;
485
 
      }
486
 
      break;
487
 
    case 132:                   /* --debug */
 
433
    case 128:
 
434
      plugindir = arg;
 
435
      break;
 
436
    case 129:
 
437
      uid = (uid_t)strtol(arg, NULL, 10);
 
438
      break;
 
439
    case 130:
 
440
      gid = (gid_t)strtol(arg, NULL, 10);
 
441
      break;
 
442
    case 131:
488
443
      debug = true;
489
444
      break;
490
 
/*
491
 
 * When adding more options before this line, remember to also add a
492
 
 * "case" to the "parse_opt_config_file" function below.
493
 
 */
494
 
    case ARGP_KEY_ARG:
495
 
      /* Cryptsetup always passes an argument, which is an empty
496
 
         string if "none" was specified in /etc/crypttab.  So if
497
 
         argument was empty, we ignore it silently. */
498
 
      if(arg[0] != '\0'){
499
 
        fprintf(stderr, "Ignoring unknown argument \"%s\"\n", arg);
500
 
      }
501
 
      break;
502
 
    case ARGP_KEY_END:
503
 
      break;
504
 
    default:
505
 
      return ARGP_ERR_UNKNOWN;
506
 
    }
507
 
    return 0;
508
 
  }
509
 
  
510
 
  /* This option parser is the same as parse_opt() above, except it
511
 
     ignores everything but the --config-file option. */
512
 
  error_t parse_opt_config_file(int key, char *arg,
513
 
                                __attribute__((unused))
514
 
                                struct argp_state *state){
515
 
    switch(key){
516
 
    case 'g':                   /* --global-options */
517
 
    case 'G':                   /* --global-env */
518
 
    case 'o':                   /* --options-for */
519
 
    case 'E':                   /* --env-for */
520
 
    case 'd':                   /* --disable */
521
 
    case 'e':                   /* --enable */
522
 
    case 128:                   /* --plugin-dir */
523
 
      break;
524
 
    case 129:                   /* --config-file */
525
 
      free(argfile);
526
 
      argfile = strdup(arg);
527
 
      if(argfile == NULL){
528
 
        perror("strdup");
529
 
      }
530
 
      break;      
531
 
    case 130:                   /* --userid */
532
 
    case 131:                   /* --groupid */
533
 
    case 132:                   /* --debug */
534
 
    case ARGP_KEY_ARG:
535
 
    case ARGP_KEY_END:
536
 
      break;
537
 
    default:
538
 
      return ARGP_ERR_UNKNOWN;
539
 
    }
540
 
    return 0;
541
 
  }
542
 
  
543
 
  struct argp argp = { .options = options,
544
 
                       .parser = parse_opt_config_file,
545
 
                       .args_doc = "",
 
445
    case ARGP_KEY_ARG:
 
446
      fprintf(stderr, "Ignoring unknown argument \"%s\"\n", arg);
 
447
      break;
 
448
    case ARGP_KEY_END:
 
449
      break;
 
450
    default:
 
451
      return ARGP_ERR_UNKNOWN;
 
452
    }
 
453
    return 0;
 
454
  }
 
455
  
 
456
  plugin *plugin_list = NULL;
 
457
  
 
458
  struct argp argp = { .options = options, .parser = parse_opt,
 
459
                       .args_doc = "[+PLUS_SEPARATED_OPTIONS]",
546
460
                       .doc = "Mandos plugin runner -- Run plugins" };
547
461
  
548
 
  /* Parse using parse_opt_config_file() in order to get the custom
549
 
     config file location, if any. */
550
 
  ret = argp_parse(&argp, argc, argv, ARGP_IN_ORDER, 0, NULL);
551
 
  if(ret == ARGP_ERR_UNKNOWN){
 
462
  ret = argp_parse (&argp, argc, argv, 0, 0, &plugin_list);
 
463
  if (ret == ARGP_ERR_UNKNOWN){
552
464
    fprintf(stderr, "Unknown error while parsing arguments\n");
553
465
    exitstatus = EXIT_FAILURE;
554
466
    goto fallback;
555
467
  }
556
 
  
557
 
  /* Reset to the normal argument parser */
558
 
  argp.parser = parse_opt;
559
 
  
560
 
  /* Open the configfile if available */
561
 
  if(argfile == NULL){
562
 
    conffp = fopen(AFILE, "r");
563
 
  } else {
564
 
    conffp = fopen(argfile, "r");
565
 
  }  
 
468
 
 
469
  conffp = fopen(argfile, "r");
566
470
  if(conffp != NULL){
567
471
    char *org_line = NULL;
568
472
    char *p, *arg, *new_arg, *line;
569
473
    size_t size = 0;
 
474
    ssize_t sret;
570
475
    const char whitespace_delims[] = " \r\t\f\v\n";
571
476
    const char comment_delim[] = "#";
572
 
    
573
 
    custom_argc = 1;
574
 
    custom_argv = malloc(sizeof(char*) * 2);
575
 
    if(custom_argv == NULL){
576
 
      perror("malloc");
577
 
      exitstatus = EXIT_FAILURE;
578
 
      goto fallback;
579
 
    }
580
 
    custom_argv[0] = argv[0];
581
 
    custom_argv[1] = NULL;
582
 
    
583
 
    /* for each line in the config file, strip whitespace and ignore
584
 
       commented text */
 
477
 
585
478
    while(true){
586
479
      sret = getline(&org_line, &size, conffp);
587
480
      if(sret == -1){
588
481
        break;
589
482
      }
590
 
      
 
483
 
591
484
      line = org_line;
592
485
      arg = strsep(&line, comment_delim);
593
486
      while((p = strsep(&arg, whitespace_delims)) != NULL){
595
488
          continue;
596
489
        }
597
490
        new_arg = strdup(p);
598
 
        if(new_arg == NULL){
599
 
          perror("strdup");
600
 
          exitstatus = EXIT_FAILURE;
601
 
          free(org_line);
602
 
          goto fallback;
603
 
        }
604
 
        
605
 
        custom_argc += 1;
606
 
        custom_argv = realloc(custom_argv, sizeof(char *)
607
 
                              * ((unsigned int) custom_argc + 1));
608
 
        if(custom_argv == NULL){
609
 
          perror("realloc");
610
 
          exitstatus = EXIT_FAILURE;
611
 
          free(org_line);
612
 
          goto fallback;
613
 
        }
614
 
        custom_argv[custom_argc-1] = new_arg;
615
 
        custom_argv[custom_argc] = NULL;        
 
491
        custom_argv = add_to_argv(custom_argv, &custom_argc, new_arg);
 
492
        if (custom_argv == NULL){
 
493
          perror("add_to_argv");
 
494
          exitstatus = EXIT_FAILURE;
 
495
          goto fallback;
 
496
        }
616
497
      }
617
498
    }
618
499
    free(org_line);
619
 
  } else {
 
500
  } else{
620
501
    /* Check for harmful errors and go to fallback. Other errors might
621
502
       not affect opening plugins */
622
 
    if(errno == EMFILE or errno == ENFILE or errno == ENOMEM){
 
503
    if (errno == EMFILE or errno == ENFILE or errno == ENOMEM){
623
504
      perror("fopen");
624
505
      exitstatus = EXIT_FAILURE;
625
506
      goto fallback;
626
507
    }
627
508
  }
628
 
  /* If there was any arguments from configuration file,
629
 
     pass them to parser as command arguments */
 
509
 
630
510
  if(custom_argv != NULL){
631
 
    ret = argp_parse(&argp, custom_argc, custom_argv, ARGP_IN_ORDER,
632
 
                     0, NULL);
633
 
    if(ret == ARGP_ERR_UNKNOWN){
 
511
    custom_argv[0] = argv[0];
 
512
    ret = argp_parse (&argp, custom_argc, custom_argv, 0, 0, &plugin_list);
 
513
    if (ret == ARGP_ERR_UNKNOWN){
634
514
      fprintf(stderr, "Unknown error while parsing arguments\n");
635
515
      exitstatus = EXIT_FAILURE;
636
516
      goto fallback;
637
517
    }
638
518
  }
639
519
  
640
 
  /* Parse actual command line arguments, to let them override the
641
 
     config file */
642
 
  ret = argp_parse(&argp, argc, argv, ARGP_IN_ORDER, 0, NULL);
643
 
  if(ret == ARGP_ERR_UNKNOWN){
644
 
    fprintf(stderr, "Unknown error while parsing arguments\n");
645
 
    exitstatus = EXIT_FAILURE;
646
 
    goto fallback;
647
 
  }
648
 
  
649
520
  if(debug){
650
521
    for(plugin *p = plugin_list; p != NULL; p=p->next){
651
522
      fprintf(stderr, "Plugin: %s has %d arguments\n",
653
524
      for(char **a = p->argv; *a != NULL; a++){
654
525
        fprintf(stderr, "\tArg: %s\n", *a);
655
526
      }
656
 
      fprintf(stderr, "...and %d environment variables\n", p->envc);
 
527
      fprintf(stderr, "...and %u environment variables\n", p->envc);
657
528
      for(char **a = p->environ; *a != NULL; a++){
658
529
        fprintf(stderr, "\t%s\n", *a);
659
530
      }
660
531
    }
661
532
  }
662
533
  
663
 
  /* Strip permissions down to nobody */
 
534
  ret = setuid(uid);
 
535
  if (ret == -1){
 
536
    perror("setuid");
 
537
  }
 
538
  
664
539
  setgid(gid);
665
 
  if(ret == -1){
 
540
  if (ret == -1){
666
541
    perror("setgid");
667
542
  }
668
 
  ret = setuid(uid);
669
 
  if(ret == -1){
670
 
    perror("setuid");
671
 
  }
672
 
  
673
 
  if(plugindir == NULL){
674
 
    dir = opendir(PDIR);
675
 
  } else {
676
 
    dir = opendir(plugindir);
677
 
  }
678
 
  
 
543
  
 
544
  dir = opendir(plugindir);
679
545
  if(dir == NULL){
680
546
    perror("Could not open plugin dir");
681
547
    exitstatus = EXIT_FAILURE;
697
563
  
698
564
  FD_ZERO(&rfds_all);
699
565
  
700
 
  /* Read and execute any executable in the plugin directory*/
701
566
  while(true){
702
567
    dirst = readdir(dir);
703
568
    
704
 
    /* All directory entries have been processed */
 
569
    // All directory entries have been processed
705
570
    if(dirst == NULL){
706
 
      if(errno == EBADF){
 
571
      if (errno == EBADF){
707
572
        perror("readdir");
708
573
        exitstatus = EXIT_FAILURE;
709
574
        goto fallback;
713
578
    
714
579
    d_name_len = strlen(dirst->d_name);
715
580
    
716
 
    /* Ignore dotfiles, backup files and other junk */
 
581
    // Ignore dotfiles, backup files and other junk
717
582
    {
718
583
      bool bad_name = false;
719
584
      
721
586
      
722
587
      const char const *bad_suffixes[] = { "~", "#", ".dpkg-new",
723
588
                                           ".dpkg-old",
724
 
                                           ".dpkg-bak",
725
589
                                           ".dpkg-divert", NULL };
726
590
      for(const char **pre = bad_prefixes; *pre != NULL; pre++){
727
591
        size_t pre_len = strlen(*pre);
735
599
          break;
736
600
        }
737
601
      }
 
602
      
738
603
      if(bad_name){
739
604
        continue;
740
605
      }
 
606
      
741
607
      for(const char **suf = bad_suffixes; *suf != NULL; suf++){
742
608
        size_t suf_len = strlen(*suf);
743
609
        if((d_name_len >= suf_len)
756
622
        continue;
757
623
      }
758
624
    }
759
 
    
 
625
 
760
626
    char *filename;
761
 
    if(plugindir == NULL){
762
 
      ret = asprintf(&filename, PDIR "/%s", dirst->d_name);
763
 
    } else {
764
 
      ret = asprintf(&filename, "%s/%s", plugindir, dirst->d_name);
765
 
    }
 
627
    ret = asprintf(&filename, "%s/%s", plugindir, dirst->d_name);
766
628
    if(ret < 0){
767
629
      perror("asprintf");
768
630
      continue;
769
631
    }
770
632
    
771
633
    ret = stat(filename, &st);
772
 
    if(ret == -1){
 
634
    if (ret == -1){
773
635
      perror("stat");
774
636
      free(filename);
775
637
      continue;
776
638
    }
777
639
    
778
 
    /* Ignore non-executable files */
779
 
    if(not S_ISREG(st.st_mode) or (access(filename, X_OK) != 0)){
 
640
    if (not S_ISREG(st.st_mode) or (access(filename, X_OK) != 0)){
780
641
      if(debug){
781
642
        fprintf(stderr, "Ignoring plugin dir entry \"%s\""
782
643
                " with bad type or mode\n", filename);
784
645
      free(filename);
785
646
      continue;
786
647
    }
787
 
    
788
 
    plugin *p = getplugin(dirst->d_name);
 
648
    plugin *p = getplugin(dirst->d_name, &plugin_list);
789
649
    if(p == NULL){
790
650
      perror("getplugin");
791
651
      free(filename);
801
661
    }
802
662
    {
803
663
      /* Add global arguments to argument list for this plugin */
804
 
      plugin *g = getplugin(NULL);
 
664
      plugin *g = getplugin(NULL, &plugin_list);
805
665
      if(g != NULL){
806
666
        for(char **a = g->argv + 1; *a != NULL; a++){
807
667
          if(not add_argument(p, *a)){
810
670
        }
811
671
        /* Add global environment variables */
812
672
        for(char **e = g->environ; *e != NULL; e++){
813
 
          if(not add_environment(p, *e, false)){
 
673
          if(not add_environment(p, *e)){
814
674
            perror("add_environment");
815
675
          }
816
676
        }
821
681
       process, too. */
822
682
    if(p->environ[0] != NULL){
823
683
      for(char **e = environ; *e != NULL; e++){
824
 
        if(not add_environment(p, *e, false)){
 
684
        char *copy = strdup(*e);
 
685
        if(copy == NULL){
 
686
          perror("strdup");
 
687
          continue;
 
688
        }
 
689
        if(not add_environment(p, copy)){
825
690
          perror("add_environment");
826
691
        }
827
692
      }
829
694
    
830
695
    int pipefd[2];
831
696
    ret = pipe(pipefd);
832
 
    if(ret == -1){
 
697
    if (ret == -1){
833
698
      perror("pipe");
834
699
      exitstatus = EXIT_FAILURE;
835
700
      goto fallback;
836
701
    }
837
 
    /* Ask OS to automatic close the pipe on exec */
838
702
    ret = set_cloexec_flag(pipefd[0]);
839
703
    if(ret < 0){
840
704
      perror("set_cloexec_flag");
848
712
      goto fallback;
849
713
    }
850
714
    /* Block SIGCHLD until process is safely in process list */
851
 
    ret = sigprocmask(SIG_BLOCK, &sigchld_action.sa_mask, NULL);
 
715
    ret = sigprocmask (SIG_BLOCK, &sigchld_action.sa_mask, NULL);
852
716
    if(ret < 0){
853
717
      perror("sigprocmask");
854
718
      exitstatus = EXIT_FAILURE;
855
719
      goto fallback;
856
720
    }
857
 
    /* Starting a new process to be watched */
 
721
    // Starting a new process to be watched
858
722
    pid_t pid = fork();
859
723
    if(pid == -1){
860
724
      perror("fork");
868
732
        perror("sigaction");
869
733
        _exit(EXIT_FAILURE);
870
734
      }
871
 
      ret = sigprocmask(SIG_UNBLOCK, &sigchld_action.sa_mask, NULL);
 
735
      ret = sigprocmask (SIG_UNBLOCK, &sigchld_action.sa_mask, NULL);
872
736
      if(ret < 0){
873
737
        perror("sigprocmask");
874
738
        _exit(EXIT_FAILURE);
875
739
      }
876
 
      
 
740
 
877
741
      ret = dup2(pipefd[1], STDOUT_FILENO); /* replace our stdout */
878
742
      if(ret == -1){
879
743
        perror("dup2");
898
762
      }
899
763
      /* no return */
900
764
    }
901
 
    /* Parent process */
902
 
    close(pipefd[1]);           /* Close unused write end of pipe */
 
765
    /* parent process */
903
766
    free(filename);
904
 
    plugin *new_plugin = getplugin(dirst->d_name);
905
 
    if(new_plugin == NULL){
906
 
      perror("getplugin");
907
 
      ret = sigprocmask(SIG_UNBLOCK, &sigchld_action.sa_mask, NULL);
 
767
    close(pipefd[1]);           /* close unused write end of pipe */
 
768
    process *new_process = malloc(sizeof(process));
 
769
    if (new_process == NULL){
 
770
      perror("malloc");
 
771
      ret = sigprocmask (SIG_UNBLOCK, &sigchld_action.sa_mask, NULL);
908
772
      if(ret < 0){
909
 
        perror("sigprocmask");
 
773
        perror("sigprocmask");
910
774
      }
911
775
      exitstatus = EXIT_FAILURE;
912
776
      goto fallback;
913
777
    }
914
778
    
915
 
    new_plugin->pid = pid;
916
 
    new_plugin->fd = pipefd[0];
917
 
    
 
779
    *new_process = (struct process){ .pid = pid,
 
780
                                     .fd = pipefd[0],
 
781
                                     .next = process_list };
 
782
    // List handling
 
783
    process_list = new_process;
918
784
    /* Unblock SIGCHLD so signal handler can be run if this process
919
785
       has already completed */
920
 
    ret = sigprocmask(SIG_UNBLOCK, &sigchld_action.sa_mask, NULL);
 
786
    ret = sigprocmask (SIG_UNBLOCK, &sigchld_action.sa_mask, NULL);
921
787
    if(ret < 0){
922
788
      perror("sigprocmask");
923
789
      exitstatus = EXIT_FAILURE;
924
790
      goto fallback;
925
791
    }
926
792
    
927
 
    FD_SET(new_plugin->fd, &rfds_all);
 
793
    FD_SET(new_process->fd, &rfds_all);
928
794
    
929
 
    if(maxfd < new_plugin->fd){
930
 
      maxfd = new_plugin->fd;
 
795
    if (maxfd < new_process->fd){
 
796
      maxfd = new_process->fd;
931
797
    }
 
798
    
932
799
  }
933
800
  
 
801
  free_plugin_list(plugin_list);
 
802
  plugin_list = NULL;
 
803
  
934
804
  closedir(dir);
935
805
  dir = NULL;
936
 
  free_plugin(getplugin(NULL));
937
 
  
938
 
  for(plugin *p = plugin_list; p != NULL; p = p->next){
939
 
    if(p->pid != 0){
940
 
      break;
941
 
    }
942
 
    if(p->next == NULL){
943
 
      fprintf(stderr, "No plugin processes started. Incorrect plugin"
944
 
              " directory?\n");
945
 
      free_plugin_list();
946
 
    }
 
806
    
 
807
  if (process_list == NULL){
 
808
    fprintf(stderr, "No plugin processes started. Incorrect plugin"
 
809
            " directory?\n");
 
810
    process_list = NULL;
947
811
  }
948
 
  
949
 
  /* Main loop while running plugins exist */
950
 
  while(plugin_list){
 
812
  while(process_list){
951
813
    fd_set rfds = rfds_all;
952
814
    int select_ret = select(maxfd+1, &rfds, NULL, NULL, NULL);
953
 
    if(select_ret == -1){
 
815
    if (select_ret == -1){
954
816
      perror("select");
955
817
      exitstatus = EXIT_FAILURE;
956
818
      goto fallback;
957
819
    }
958
820
    /* OK, now either a process completed, or something can be read
959
821
       from one of them */
960
 
    for(plugin *proc = plugin_list; proc != NULL;){
 
822
    for(process *proc = process_list; proc ; proc = proc->next){
961
823
      /* Is this process completely done? */
962
 
      if(proc->completed and proc->eof){
 
824
      if(proc->eof and proc->completed){
963
825
        /* Only accept the plugin output if it exited cleanly */
964
826
        if(not WIFEXITED(proc->status)
965
827
           or WEXITSTATUS(proc->status) != 0){
966
828
          /* Bad exit by plugin */
967
 
          
968
829
          if(debug){
969
830
            if(WIFEXITED(proc->status)){
970
 
              fprintf(stderr, "Plugin %s [%" PRIdMAX "] exited with"
971
 
                      " status %d\n", proc->name,
972
 
                      (intmax_t) (proc->pid),
 
831
              fprintf(stderr, "Plugin %u exited with status %d\n",
 
832
                      (unsigned int) (proc->pid),
973
833
                      WEXITSTATUS(proc->status));
974
 
            } else if(WIFSIGNALED(proc->status)){
975
 
              fprintf(stderr, "Plugin %s [%" PRIdMAX "] killed by"
976
 
                      " signal %d\n", proc->name,
977
 
                      (intmax_t) (proc->pid),
 
834
            } else if(WIFSIGNALED(proc->status)) {
 
835
              fprintf(stderr, "Plugin %u killed by signal %d\n",
 
836
                      (unsigned int) (proc->pid),
978
837
                      WTERMSIG(proc->status));
979
838
            } else if(WCOREDUMP(proc->status)){
980
 
              fprintf(stderr, "Plugin %s [%" PRIdMAX "] dumped"
981
 
                      " core\n", proc->name, (intmax_t) (proc->pid));
 
839
              fprintf(stderr, "Plugin %d dumped core\n",
 
840
                      (unsigned int) (proc->pid));
982
841
            }
983
842
          }
984
 
          
985
843
          /* Remove the plugin */
986
844
          FD_CLR(proc->fd, &rfds_all);
987
 
          
988
845
          /* Block signal while modifying process_list */
989
 
          ret = sigprocmask(SIG_BLOCK, &sigchld_action.sa_mask, NULL);
 
846
          ret = sigprocmask (SIG_BLOCK, &sigchld_action.sa_mask, NULL);
990
847
          if(ret < 0){
991
848
            perror("sigprocmask");
992
849
            exitstatus = EXIT_FAILURE;
993
850
            goto fallback;
994
851
          }
995
 
          
996
 
          plugin *next_plugin = proc->next;
997
 
          free_plugin(proc);
998
 
          proc = next_plugin;
999
 
          
 
852
          /* Delete this process entry from the list */
 
853
          if(process_list == proc){
 
854
            /* First one - simple */
 
855
            process_list = proc->next;
 
856
          } else {
 
857
            /* Second one or later */
 
858
            for(process *p = process_list; p != NULL; p = p->next){
 
859
              if(p->next == proc){
 
860
                p->next = proc->next;
 
861
                break;
 
862
              }
 
863
            }
 
864
          }
1000
865
          /* We are done modifying process list, so unblock signal */
1001
 
          ret = sigprocmask(SIG_UNBLOCK, &sigchld_action.sa_mask,
1002
 
                            NULL);
 
866
          ret = sigprocmask (SIG_UNBLOCK, &sigchld_action.sa_mask,
 
867
                             NULL);
1003
868
          if(ret < 0){
1004
869
            perror("sigprocmask");
1005
 
            exitstatus = EXIT_FAILURE;
1006
 
            goto fallback;
1007
 
          }
1008
 
          
1009
 
          if(plugin_list == NULL){
1010
 
            break;
1011
 
          }
1012
 
          
1013
 
          continue;
 
870
          }
 
871
          free(proc->buffer);
 
872
          free(proc);
 
873
          /* We deleted this process from the list, so we can't go
 
874
             proc->next.  Therefore, start over from the beginning of
 
875
             the process list */
 
876
          break;
1014
877
        }
1015
 
        
1016
878
        /* This process exited nicely, so print its buffer */
1017
 
        
1018
 
        bool bret = print_out_password(proc->buffer,
1019
 
                                       proc->buffer_length);
 
879
 
 
880
        bool bret = print_out_password(proc->buffer, proc->buffer_length);
1020
881
        if(not bret){
1021
882
          perror("print_out_password");
1022
883
          exitstatus = EXIT_FAILURE;
1023
884
        }
1024
885
        goto fallback;
1025
886
      }
1026
 
      
1027
887
      /* This process has not completed.  Does it have any output? */
1028
888
      if(proc->eof or not FD_ISSET(proc->fd, &rfds)){
1029
889
        /* This process had nothing to say at this time */
1030
 
        proc = proc->next;
1031
890
        continue;
1032
891
      }
1033
892
      /* Before reading, make the process' data buffer large enough */
1034
893
      if(proc->buffer_length + BUFFER_SIZE > proc->buffer_size){
1035
894
        proc->buffer = realloc(proc->buffer, proc->buffer_size
1036
895
                               + (size_t) BUFFER_SIZE);
1037
 
        if(proc->buffer == NULL){
 
896
        if (proc->buffer == NULL){
1038
897
          perror("malloc");
1039
898
          exitstatus = EXIT_FAILURE;
1040
899
          goto fallback;
1042
901
        proc->buffer_size += BUFFER_SIZE;
1043
902
      }
1044
903
      /* Read from the process */
1045
 
      sret = read(proc->fd, proc->buffer + proc->buffer_length,
1046
 
                  BUFFER_SIZE);
1047
 
      if(sret < 0){
 
904
      ret = read(proc->fd, proc->buffer + proc->buffer_length,
 
905
                 BUFFER_SIZE);
 
906
      if(ret < 0){
1048
907
        /* Read error from this process; ignore the error */
1049
 
        proc = proc->next;
1050
908
        continue;
1051
909
      }
1052
 
      if(sret == 0){
 
910
      if(ret == 0){
1053
911
        /* got EOF */
1054
912
        proc->eof = true;
1055
913
      } else {
1056
 
        proc->buffer_length += (size_t) sret;
 
914
        proc->buffer_length += (size_t) ret;
1057
915
      }
1058
916
    }
1059
917
  }
1060
 
  
1061
 
  
 
918
 
 
919
 
1062
920
 fallback:
1063
921
  
1064
 
  if(plugin_list == NULL or exitstatus != EXIT_SUCCESS){
1065
 
    /* Fallback if all plugins failed, none are found or an error
1066
 
       occured */
 
922
  if(process_list == NULL or exitstatus != EXIT_SUCCESS){
 
923
    /* Fallback if all plugins failed, none are found or an error occured */
1067
924
    bool bret;
1068
925
    fprintf(stderr, "Going to fallback mode using getpass(3)\n");
1069
926
    char *passwordbuffer = getpass("Password: ");
1070
 
    size_t len = strlen(passwordbuffer);
1071
 
    /* Strip trailing newline */
1072
 
    if(len > 0 and passwordbuffer[len-1] == '\n'){
1073
 
      passwordbuffer[len-1] = '\0'; /* not strictly necessary */
1074
 
      len--;
1075
 
    }
1076
 
    bret = print_out_password(passwordbuffer, len);
 
927
    bret = print_out_password(passwordbuffer, strlen(passwordbuffer));
1077
928
    if(not bret){
1078
929
      perror("print_out_password");
1079
930
      exitstatus = EXIT_FAILURE;
1086
937
    perror("sigaction");
1087
938
    exitstatus = EXIT_FAILURE;
1088
939
  }
1089
 
  
 
940
 
1090
941
  if(custom_argv != NULL){
1091
 
    for(char **arg = custom_argv+1; *arg != NULL; arg++){
 
942
    for(char **arg = custom_argv; *arg != NULL; arg++){
1092
943
      free(*arg);
1093
944
    }
1094
945
    free(custom_argv);
1095
946
  }
 
947
  free_plugin_list(plugin_list);
1096
948
  
1097
949
  if(dir != NULL){
1098
950
    closedir(dir);
1099
951
  }
1100
952
  
1101
 
  /* Kill the processes */
1102
 
  for(plugin *p = plugin_list; p != NULL; p = p->next){
1103
 
    if(p->pid != 0){
1104
 
      close(p->fd);
1105
 
      ret = kill(p->pid, SIGTERM);
1106
 
      if(ret == -1 and errno != ESRCH){
1107
 
        /* Set-uid proccesses might not get closed */
1108
 
        perror("kill");
1109
 
      }
 
953
  /* Free the process list and kill the processes */
 
954
  for(process *next; process_list != NULL; process_list = next){
 
955
    next = process_list->next;
 
956
    close(process_list->fd);
 
957
    ret = kill(process_list->pid, SIGTERM);
 
958
    if(ret == -1 and errno != ESRCH){
 
959
      /* set-uid proccesses migth not get closed */
 
960
      perror("kill");
1110
961
    }
 
962
    free(process_list->buffer);
 
963
    free(process_list);
1111
964
  }
1112
965
  
1113
966
  /* Wait for any remaining child processes to terminate */
1118
971
    perror("wait");
1119
972
  }
1120
973
  
1121
 
  free_plugin_list();
1122
 
  
1123
 
  free(plugindir);
1124
 
  free(argfile);
1125
 
  
1126
974
  return exitstatus;
1127
975
}