/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 at recompile
  • Date: 2020-04-05 21:30:59 UTC
  • Revision ID: teddy@recompile.se-20200405213059-fb2a61ckqynrmatk
Fix file descriptor leak in mandos-client

When the local network has Mandos servers announcing themselves using
real, globally reachable, IPv6 addresses (i.e. not link-local
addresses), but there is no router on the local network providing IPv6
RA (Router Advertisement) packets, the client cannot reach the server
by normal means, since the client only has a link-local IPv6 address,
and has no usable route to reach the server's global IPv6 address.
(This is not a common situation, and usually only happens when the
router itself reboots and runs a Mandos client, since it cannot then
give RA packets to itself.)  The client code has a solution for
this, which consists of adding a temporary local route to reach the
address of the server during communication, and removing this
temporary route afterwards.

This solution with a temporary route works, but has a file descriptor
leak; it leaks one file descriptor for each addition and for each
removal of a route.  If one server requiring an added route is present
on the network, but no servers gives a password, making the client
retry after the default ten seconds, and we furthermore assume a
default 1024 open files limit, the client runs out of file descriptors
after about 90 minutes, after which time the client process will be
useless and fail to retrieve any passwords, necessitating manual
password entry via the keyboard.

Fix this by eliminating the file descriptor leak in the client.

* plugins.d/mandos-client.c (add_delete_local_route): Do
  close(devnull) also in parent process, also if fork() fails, and on
  any failure in child process.

Show diffs side-by-side

added added

removed removed

Lines of Context:
1
 
/*  -*- coding: utf-8 -*- */
 
1
/*  -*- coding: utf-8; mode: c; mode: orgtbl -*- */
2
2
/*
3
3
 * Mandos plugin runner - Run Mandos plugins
4
4
 *
5
 
 * Copyright © 2007-2008 Teddy Hogeborn & Björn Påhlsson
6
 
 * 
7
 
 * This program is free software: you can redistribute it and/or
8
 
 * modify it under the terms of the GNU General Public License as
9
 
 * published by the Free Software Foundation, either version 3 of the
10
 
 * License, or (at your option) any later version.
11
 
 * 
12
 
 * This program is distributed in the hope that it will be useful, but
 
5
 * Copyright © 2008-2018 Teddy Hogeborn
 
6
 * Copyright © 2008-2018 Björn Påhlsson
 
7
 * 
 
8
 * This file is part of Mandos.
 
9
 * 
 
10
 * Mandos is free software: you can redistribute it and/or modify it
 
11
 * under the terms of the GNU General Public License as published by
 
12
 * the Free Software Foundation, either version 3 of the License, or
 
13
 * (at your option) any later version.
 
14
 * 
 
15
 * Mandos is distributed in the hope that it will be useful, but
13
16
 * WITHOUT ANY WARRANTY; without even the implied warranty of
14
17
 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
15
18
 * General Public License for more details.
16
19
 * 
17
20
 * You should have received a copy of the GNU General Public License
18
 
 * along with this program.  If not, see
19
 
 * <http://www.gnu.org/licenses/>.
 
21
 * along with Mandos.  If not, see <http://www.gnu.org/licenses/>.
20
22
 * 
21
 
 * Contact the authors at <mandos@fukt.bsnet.se>.
 
23
 * Contact the authors at <mandos@recompile.se>.
22
24
 */
23
25
 
24
26
#define _GNU_SOURCE             /* TEMP_FAILURE_RETRY(), getline(),
25
 
                                   asprintf() */
 
27
                                   O_CLOEXEC, pipe2() */
26
28
#include <stddef.h>             /* size_t, NULL */
27
 
#include <stdlib.h>             /* malloc(), exit(), EXIT_FAILURE,
28
 
                                   EXIT_SUCCESS, realloc() */
 
29
#include <stdlib.h>             /* malloc(), reallocarray(), realloc(),
 
30
                                   EXIT_SUCCESS, exit() */
29
31
#include <stdbool.h>            /* bool, true, false */
30
 
#include <stdio.h>              /* perror, popen(), fileno(),
31
 
                                   fprintf(), stderr, STDOUT_FILENO */
32
 
#include <sys/types.h>          /* DIR, opendir(), stat(), struct
33
 
                                   stat, waitpid(), WIFEXITED(),
34
 
                                   WEXITSTATUS(), wait(), pid_t,
35
 
                                   uid_t, gid_t, getuid(), getgid(),
36
 
                                   dirfd() */
 
32
#include <stdio.h>              /* fileno(), fprintf(),
 
33
                                   stderr, STDOUT_FILENO, fclose() */
 
34
#include <sys/types.h>          /* fstat(), struct stat, waitpid(),
 
35
                                   WIFEXITED(), WEXITSTATUS(), wait(),
 
36
                                   pid_t, uid_t, gid_t, getuid(),
 
37
                                   getgid() */
37
38
#include <sys/select.h>         /* fd_set, select(), FD_ZERO(),
38
39
                                   FD_SET(), FD_ISSET(), FD_CLR */
39
40
#include <sys/wait.h>           /* wait(), waitpid(), WIFEXITED(),
40
 
                                   WEXITSTATUS() */
41
 
#include <sys/stat.h>           /* struct stat, stat(), S_ISREG() */
 
41
                                   WEXITSTATUS(), WTERMSIG() */
 
42
#include <sys/stat.h>           /* struct stat, fstat(), S_ISREG() */
42
43
#include <iso646.h>             /* and, or, not */
43
 
#include <dirent.h>             /* DIR, struct dirent, opendir(),
44
 
                                   readdir(), closedir(), dirfd() */
45
 
#include <unistd.h>             /* struct stat, stat(), S_ISREG(),
46
 
                                   fcntl(), setuid(), setgid(),
47
 
                                   F_GETFD, F_SETFD, FD_CLOEXEC,
48
 
                                   access(), pipe(), fork(), close()
49
 
                                   dup2, STDOUT_FILENO, _exit(),
50
 
                                   execv(), write(), read(),
51
 
                                   close() */
 
44
#include <dirent.h>             /* struct dirent, scandirat() */
 
45
#include <unistd.h>             /* fcntl(), F_GETFD, F_SETFD,
 
46
                                   FD_CLOEXEC, write(), STDOUT_FILENO,
 
47
                                   struct stat, fstat(), close(),
 
48
                                   setgid(), setuid(), S_ISREG(),
 
49
                                   faccessat() pipe2(), fork(),
 
50
                                   _exit(), dup2(), fexecve(), read()
 
51
                                */
52
52
#include <fcntl.h>              /* fcntl(), F_GETFD, F_SETFD,
53
 
                                   FD_CLOEXEC */
54
 
#include <string.h>             /* strsep, strlen(), asprintf() */
 
53
                                   FD_CLOEXEC, openat(), scandirat(),
 
54
                                   pipe2() */
 
55
#include <string.h>             /* strsep, strlen(), strsignal(),
 
56
                                   strcmp(), strncmp() */
55
57
#include <errno.h>              /* errno */
56
58
#include <argp.h>               /* struct argp_option, struct
57
59
                                   argp_state, struct argp,
61
63
#include <signal.h>             /* struct sigaction, sigemptyset(),
62
64
                                   sigaddset(), sigaction(),
63
65
                                   sigprocmask(), SIG_BLOCK, SIGCHLD,
64
 
                                   SIG_UNBLOCK, kill() */
 
66
                                   SIG_UNBLOCK, kill(), sig_atomic_t
 
67
                                */
65
68
#include <errno.h>              /* errno, EBADF */
 
69
#include <inttypes.h>           /* intmax_t, PRIdMAX, strtoimax() */
 
70
#include <sysexits.h>           /* EX_OSERR, EX_USAGE, EX_IOERR,
 
71
                                   EX_CONFIG, EX_UNAVAILABLE, EX_OK */
 
72
#include <errno.h>              /* errno */
 
73
#include <error.h>              /* error() */
 
74
#include <fnmatch.h>            /* fnmatch() */
66
75
 
67
76
#define BUFFER_SIZE 256
68
77
 
69
78
#define PDIR "/lib/mandos/plugins.d"
 
79
#define PHDIR "/lib/mandos/plugin-helpers"
70
80
#define AFILE "/conf/conf.d/mandos/plugin-runner.conf"
71
81
 
72
 
const char *argp_program_version = "plugin-runner 1.0";
73
 
const char *argp_program_bug_address = "<mandos@fukt.bsnet.se>";
74
 
 
75
 
struct plugin;
 
82
const char *argp_program_version = "plugin-runner " VERSION;
 
83
const char *argp_program_bug_address = "<mandos@recompile.se>";
76
84
 
77
85
typedef struct plugin{
78
86
  char *name;                   /* can be NULL or any plugin name */
81
89
  char **environ;
82
90
  int envc;
83
91
  bool disabled;
84
 
 
 
92
  
85
93
  /* Variables used for running processes*/
86
94
  pid_t pid;
87
95
  int fd;
89
97
  size_t buffer_size;
90
98
  size_t buffer_length;
91
99
  bool eof;
92
 
  volatile bool completed;
93
 
  volatile int status;
 
100
  volatile sig_atomic_t completed;
 
101
  int status;
94
102
  struct plugin *next;
95
103
} plugin;
96
104
 
97
105
static plugin *plugin_list = NULL;
98
106
 
99
 
/* Gets a existing plugin based on name,
 
107
/* Gets an existing plugin based on name,
100
108
   or if none is found, creates a new one */
 
109
__attribute__((warn_unused_result))
101
110
static plugin *getplugin(char *name){
102
 
  /* Check for exiting plugin with that name */
103
 
  for (plugin *p = plugin_list; p != NULL; p = p->next){
104
 
    if ((p->name == name)
105
 
        or (p->name and name and (strcmp(p->name, name) == 0))){
 
111
  /* Check for existing plugin with that name */
 
112
  for(plugin *p = plugin_list; p != NULL; p = p->next){
 
113
    if((p->name == name)
 
114
       or (p->name and name and (strcmp(p->name, name) == 0))){
106
115
      return p;
107
116
    }
108
117
  }
109
118
  /* Create a new plugin */
110
 
  plugin *new_plugin = malloc(sizeof(plugin));
111
 
  if (new_plugin == NULL){
 
119
  plugin *new_plugin = NULL;
 
120
  do {
 
121
    new_plugin = malloc(sizeof(plugin));
 
122
  } while(new_plugin == NULL and errno == EINTR);
 
123
  if(new_plugin == NULL){
112
124
    return NULL;
113
125
  }
114
126
  char *copy_name = NULL;
115
127
  if(name != NULL){
116
 
    copy_name = strdup(name);
 
128
    do {
 
129
      copy_name = strdup(name);
 
130
    } while(copy_name == NULL and errno == EINTR);
117
131
    if(copy_name == NULL){
 
132
      int e = errno;
 
133
      free(new_plugin);
 
134
      errno = e;
118
135
      return NULL;
119
136
    }
120
137
  }
121
 
 
122
 
  *new_plugin = (plugin) { .name = copy_name,
123
 
                           .argc = 1,
124
 
                           .disabled = false,
125
 
                           .next = plugin_list };
126
 
  
127
 
  new_plugin->argv = malloc(sizeof(char *) * 2);
128
 
  if (new_plugin->argv == NULL){
 
138
  
 
139
  *new_plugin = (plugin){ .name = copy_name,
 
140
                          .argc = 1,
 
141
                          .disabled = false,
 
142
                          .next = plugin_list };
 
143
  
 
144
  do {
 
145
    new_plugin->argv = malloc(sizeof(char *) * 2);
 
146
  } while(new_plugin->argv == NULL and errno == EINTR);
 
147
  if(new_plugin->argv == NULL){
 
148
    int e = errno;
129
149
    free(copy_name);
130
150
    free(new_plugin);
 
151
    errno = e;
131
152
    return NULL;
132
153
  }
133
154
  new_plugin->argv[0] = copy_name;
134
155
  new_plugin->argv[1] = NULL;
135
 
 
136
 
  new_plugin->environ = malloc(sizeof(char *));
 
156
  
 
157
  do {
 
158
    new_plugin->environ = malloc(sizeof(char *));
 
159
  } while(new_plugin->environ == NULL and errno == EINTR);
137
160
  if(new_plugin->environ == NULL){
 
161
    int e = errno;
138
162
    free(copy_name);
139
163
    free(new_plugin->argv);
140
164
    free(new_plugin);
 
165
    errno = e;
141
166
    return NULL;
142
167
  }
143
168
  new_plugin->environ[0] = NULL;
144
 
 
 
169
  
145
170
  /* Append the new plugin to the list */
146
171
  plugin_list = new_plugin;
147
172
  return new_plugin;
148
173
}
149
174
 
150
175
/* Helper function for add_argument and add_environment */
 
176
__attribute__((nonnull, warn_unused_result))
151
177
static bool add_to_char_array(const char *new, char ***array,
152
178
                              int *len){
153
179
  /* Resize the pointed-to array to hold one more pointer */
154
 
  *array = realloc(*array, sizeof(char *)
155
 
                   * (size_t) ((*len) + 2));
 
180
  char **new_array = NULL;
 
181
  do {
 
182
#if defined(__GLIBC_PREREQ) and __GLIBC_PREREQ(2, 26)
 
183
    new_array = reallocarray(*array, (size_t)((*len) + 2),
 
184
                             sizeof(char *));
 
185
#else
 
186
    if(((size_t)((*len) + 2)) > (SIZE_MAX / sizeof(char *))){
 
187
      /* overflow */
 
188
      new_array = NULL;
 
189
      errno = ENOMEM;
 
190
    } else {
 
191
      new_array = realloc(*array, (size_t)((*len) + 2)
 
192
                          * sizeof(char *));
 
193
    }
 
194
#endif
 
195
  } while(new_array == NULL and errno == EINTR);
156
196
  /* Malloc check */
157
 
  if(*array == NULL){
 
197
  if(new_array == NULL){
158
198
    return false;
159
199
  }
 
200
  *array = new_array;
160
201
  /* Make a copy of the new string */
161
 
  char *copy = strdup(new);
 
202
  char *copy;
 
203
  do {
 
204
    copy = strdup(new);
 
205
  } while(copy == NULL and errno == EINTR);
162
206
  if(copy == NULL){
163
207
    return false;
164
208
  }
171
215
}
172
216
 
173
217
/* Add to a plugin's argument vector */
 
218
__attribute__((nonnull(2), warn_unused_result))
174
219
static bool add_argument(plugin *p, const char *arg){
175
220
  if(p == NULL){
176
221
    return false;
179
224
}
180
225
 
181
226
/* Add to a plugin's environment */
182
 
static bool add_environment(plugin *p, const char *def){
 
227
__attribute__((nonnull(2), warn_unused_result))
 
228
static bool add_environment(plugin *p, const char *def, bool replace){
183
229
  if(p == NULL){
184
230
    return false;
185
231
  }
 
232
  /* namelen = length of name of environment variable */
 
233
  size_t namelen = (size_t)(strchrnul(def, '=') - def);
 
234
  /* Search for this environment variable */
 
235
  for(char **envdef = p->environ; *envdef != NULL; envdef++){
 
236
    if(strncmp(*envdef, def, namelen + 1) == 0){
 
237
      /* It already exists */
 
238
      if(replace){
 
239
        char *new_envdef;
 
240
        do {
 
241
          new_envdef = realloc(*envdef, strlen(def) + 1);
 
242
        } while(new_envdef == NULL and errno == EINTR);
 
243
        if(new_envdef == NULL){
 
244
          return false;
 
245
        }
 
246
        *envdef = new_envdef;
 
247
        strcpy(*envdef, def);
 
248
      }
 
249
      return true;
 
250
    }
 
251
  }
186
252
  return add_to_char_array(def, &(p->environ), &(p->envc));
187
253
}
188
254
 
 
255
#ifndef O_CLOEXEC
189
256
/*
190
257
 * Based on the example in the GNU LibC manual chapter 13.13 "File
191
258
 * Descriptor Flags".
192
 
 * *Note File Descriptor Flags:(libc)Descriptor Flags.
 
259
 | [[info:libc:Descriptor%20Flags][File Descriptor Flags]] |
193
260
 */
194
 
static int set_cloexec_flag(int fd)
195
 
{
196
 
  int ret = fcntl(fd, F_GETFD, 0);
 
261
__attribute__((warn_unused_result))
 
262
static int set_cloexec_flag(int fd){
 
263
  int ret = (int)TEMP_FAILURE_RETRY(fcntl(fd, F_GETFD, 0));
197
264
  /* If reading the flags failed, return error indication now. */
198
265
  if(ret < 0){
199
266
    return ret;
200
267
  }
201
268
  /* Store modified flag word in the descriptor. */
202
 
  return fcntl(fd, F_SETFD, ret | FD_CLOEXEC);
 
269
  return (int)TEMP_FAILURE_RETRY(fcntl(fd, F_SETFD,
 
270
                                       ret | FD_CLOEXEC));
203
271
}
 
272
#endif  /* not O_CLOEXEC */
204
273
 
205
274
 
206
275
/* Mark processes as completed when they exit, and save their exit
207
276
   status. */
208
 
void handle_sigchld(__attribute__((unused)) int sig){
 
277
static void handle_sigchld(__attribute__((unused)) int sig){
 
278
  int old_errno = errno;
209
279
  while(true){
210
280
    plugin *proc = plugin_list;
211
281
    int status;
215
285
      break;
216
286
    }
217
287
    if(pid == -1){
218
 
      if (errno != ECHILD){
219
 
        perror("waitpid");
 
288
      if(errno == ECHILD){
 
289
        /* No child processes */
 
290
        break;
220
291
      }
221
 
      /* No child processes */
222
 
      break;
 
292
      error(0, errno, "waitpid");
223
293
    }
224
 
 
 
294
    
225
295
    /* A child exited, find it in process_list */
226
296
    while(proc != NULL and proc->pid != pid){
227
297
      proc = proc->next;
231
301
      continue;
232
302
    }
233
303
    proc->status = status;
234
 
    proc->completed = true;
 
304
    proc->completed = 1;
235
305
  }
 
306
  errno = old_errno;
236
307
}
237
308
 
238
309
/* Prints out a password to stdout */
239
 
bool print_out_password(const char *buffer, size_t length){
 
310
__attribute__((nonnull, warn_unused_result))
 
311
static bool print_out_password(const char *buffer, size_t length){
240
312
  ssize_t ret;
241
 
  if(length>0 and buffer[length-1] == '\n'){
242
 
    length--;
243
 
  }
244
313
  for(size_t written = 0; written < length; written += (size_t)ret){
245
314
    ret = TEMP_FAILURE_RETRY(write(STDOUT_FILENO, buffer + written,
246
315
                                   length - written));
252
321
}
253
322
 
254
323
/* Removes and free a plugin from the plugin list */
 
324
__attribute__((nonnull))
255
325
static void free_plugin(plugin *plugin_node){
256
326
  
257
 
  for(char **arg = plugin_node->argv; *arg != NULL; arg++){
 
327
  for(char **arg = (plugin_node->argv)+1; *arg != NULL; arg++){
258
328
    free(*arg);
259
329
  }
 
330
  free(plugin_node->name);
260
331
  free(plugin_node->argv);
261
332
  for(char **env = plugin_node->environ; *env != NULL; env++){
262
333
    free(*env);
263
334
  }
264
335
  free(plugin_node->environ);
265
336
  free(plugin_node->buffer);
266
 
 
 
337
  
267
338
  /* Removes the plugin from the singly-linked list */
268
339
  if(plugin_node == plugin_list){
269
340
    /* First one - simple */
289
360
 
290
361
int main(int argc, char *argv[]){
291
362
  char *plugindir = NULL;
 
363
  char *pluginhelperdir = NULL;
292
364
  char *argfile = NULL;
293
365
  FILE *conffp;
294
 
  size_t d_name_len;
295
 
  DIR *dir = NULL;
296
 
  struct dirent *dirst;
 
366
  struct dirent **direntries = NULL;
297
367
  struct stat st;
298
368
  fd_set rfds_all;
299
369
  int ret, maxfd = 0;
 
370
  ssize_t sret;
300
371
  uid_t uid = 65534;
301
372
  gid_t gid = 65534;
302
373
  bool debug = false;
306
377
                                      .sa_flags = SA_NOCLDSTOP };
307
378
  char **custom_argv = NULL;
308
379
  int custom_argc = 0;
 
380
  int dir_fd = -1;
309
381
  
310
382
  /* Establish a signal handler */
311
383
  sigemptyset(&sigchld_action.sa_mask);
312
384
  ret = sigaddset(&sigchld_action.sa_mask, SIGCHLD);
313
385
  if(ret == -1){
314
 
    perror("sigaddset");
315
 
    exitstatus = EXIT_FAILURE;
 
386
    error(0, errno, "sigaddset");
 
387
    exitstatus = EX_OSERR;
316
388
    goto fallback;
317
389
  }
318
390
  ret = sigaction(SIGCHLD, &sigchld_action, &old_sigchld_action);
319
391
  if(ret == -1){
320
 
    perror("sigaction");
321
 
    exitstatus = EXIT_FAILURE;
 
392
    error(0, errno, "sigaction");
 
393
    exitstatus = EX_OSERR;
322
394
    goto fallback;
323
395
  }
324
396
  
327
399
    { .name = "global-options", .key = 'g',
328
400
      .arg = "OPTION[,OPTION[,...]]",
329
401
      .doc = "Options passed to all plugins" },
330
 
    { .name = "global-envs", .key = 'e',
 
402
    { .name = "global-env", .key = 'G',
331
403
      .arg = "VAR=value",
332
404
      .doc = "Environment variable passed to all plugins" },
333
405
    { .name = "options-for", .key = 'o',
334
406
      .arg = "PLUGIN:OPTION[,OPTION[,...]]",
335
407
      .doc = "Options passed only to specified plugin" },
336
 
    { .name = "envs-for", .key = 'f',
 
408
    { .name = "env-for", .key = 'E',
337
409
      .arg = "PLUGIN:ENV=value",
338
410
      .doc = "Environment variable passed to specified plugin" },
339
411
    { .name = "disable", .key = 'd',
340
412
      .arg = "PLUGIN",
341
413
      .doc = "Disable a specific plugin", .group = 1 },
 
414
    { .name = "enable", .key = 'e',
 
415
      .arg = "PLUGIN",
 
416
      .doc = "Enable a specific plugin", .group = 1 },
342
417
    { .name = "plugin-dir", .key = 128,
343
418
      .arg = "DIRECTORY",
344
419
      .doc = "Specify a different plugin directory", .group = 2 },
353
428
      .doc = "Group ID the plugins will run as", .group = 3 },
354
429
    { .name = "debug", .key = 132,
355
430
      .doc = "Debug mode", .group = 4 },
 
431
    { .name = "plugin-helper-dir", .key = 133,
 
432
      .arg = "DIRECTORY",
 
433
      .doc = "Specify a different plugin helper directory",
 
434
      .group = 2 },
 
435
    /*
 
436
     * These reproduce what we would get without ARGP_NO_HELP
 
437
     */
 
438
    { .name = "help", .key = '?',
 
439
      .doc = "Give this help list", .group = -1 },
 
440
    { .name = "usage", .key = -3,
 
441
      .doc = "Give a short usage message", .group = -1 },
 
442
    { .name = "version", .key = 'V',
 
443
      .doc = "Print program version", .group = -1 },
356
444
    { .name = NULL }
357
445
  };
358
446
  
359
 
  error_t parse_opt (int key, char *arg, __attribute__((unused)) struct argp_state *state) {
360
 
    /* Get the INPUT argument from `argp_parse', which we know is a
361
 
       pointer to our plugin list pointer. */
362
 
    switch (key) {
 
447
  __attribute__((nonnull(3)))
 
448
  error_t parse_opt(int key, char *arg, struct argp_state *state){
 
449
    errno = 0;
 
450
    switch(key){
 
451
      char *tmp;
 
452
      intmax_t tmp_id;
363
453
    case 'g':                   /* --global-options */
364
 
      if (arg != NULL){
365
 
        char *p;
366
 
        while((p = strsep(&arg, ",")) != NULL){
367
 
          if(p[0] == '\0'){
368
 
            continue;
369
 
          }
370
 
          if(not add_argument(getplugin(NULL), p)){
371
 
            perror("add_argument");
372
 
            return ARGP_ERR_UNKNOWN;
373
 
          }
374
 
        }
375
 
      }
376
 
      break;
377
 
    case 'e':                   /* --global-envs */
378
 
      if(arg == NULL){
379
 
        break;
380
 
      }
381
454
      {
382
 
        char *envdef = strdup(arg);
383
 
        if(envdef == NULL){
384
 
          break;
385
 
        }
386
 
        if(not add_environment(getplugin(NULL), envdef)){
387
 
          perror("add_environment");
388
 
        }
 
455
        char *plugin_option;
 
456
        while((plugin_option = strsep(&arg, ",")) != NULL){
 
457
          if(not add_argument(getplugin(NULL), plugin_option)){
 
458
            break;
 
459
          }
 
460
        }
 
461
        errno = 0;
 
462
      }
 
463
      break;
 
464
    case 'G':                   /* --global-env */
 
465
      if(add_environment(getplugin(NULL), arg, true)){
 
466
        errno = 0;
389
467
      }
390
468
      break;
391
469
    case 'o':                   /* --options-for */
392
 
      if (arg != NULL){
393
 
        char *p_name = strsep(&arg, ":");
394
 
        if(p_name[0] == '\0'){
395
 
          break;
396
 
        }
397
 
        char *opt = strsep(&arg, ":");
398
 
        if(opt[0] == '\0'){
399
 
          break;
400
 
        }
401
 
        if(opt != NULL){
402
 
          char *p;
403
 
          while((p = strsep(&opt, ",")) != NULL){
404
 
            if(p[0] == '\0'){
405
 
              continue;
406
 
            }
407
 
            if(not add_argument(getplugin(p_name), p)){
408
 
              perror("add_argument");
409
 
              return ARGP_ERR_UNKNOWN;
410
 
            }
 
470
      {
 
471
        char *option_list = strchr(arg, ':');
 
472
        if(option_list == NULL){
 
473
          argp_error(state, "No colon in \"%s\"", arg);
 
474
          errno = EINVAL;
 
475
          break;
 
476
        }
 
477
        *option_list = '\0';
 
478
        option_list++;
 
479
        if(arg[0] == '\0'){
 
480
          argp_error(state, "Empty plugin name");
 
481
          errno = EINVAL;
 
482
          break;
 
483
        }
 
484
        char *option;
 
485
        while((option = strsep(&option_list, ",")) != NULL){
 
486
          if(not add_argument(getplugin(arg), option)){
 
487
            break;
411
488
          }
412
489
        }
 
490
        errno = 0;
413
491
      }
414
492
      break;
415
 
    case 'f':                   /* --envs-for */
416
 
      if(arg == NULL){
417
 
        break;
418
 
      }
 
493
    case 'E':                   /* --env-for */
419
494
      {
420
495
        char *envdef = strchr(arg, ':');
421
496
        if(envdef == NULL){
422
 
          break;
423
 
        }
424
 
        char *p_name = strndup(arg, (size_t) (envdef-arg));
425
 
        if(p_name == NULL){
426
 
          break;
427
 
        }
 
497
          argp_error(state, "No colon in \"%s\"", arg);
 
498
          errno = EINVAL;
 
499
          break;
 
500
        }
 
501
        *envdef = '\0';
428
502
        envdef++;
429
 
        if(not add_environment(getplugin(p_name), envdef)){
430
 
          perror("add_environment");
 
503
        if(arg[0] == '\0'){
 
504
          argp_error(state, "Empty plugin name");
 
505
          errno = EINVAL;
 
506
          break;
 
507
        }
 
508
        if(add_environment(getplugin(arg), envdef, true)){
 
509
          errno = 0;
431
510
        }
432
511
      }
433
512
      break;
434
513
    case 'd':                   /* --disable */
435
 
      if (arg != NULL){
436
 
        plugin *p = getplugin(arg);
437
 
        if(p == NULL){
438
 
          return ARGP_ERR_UNKNOWN;
439
 
        }
440
 
        p->disabled = true;
 
514
      {
 
515
        plugin *p = getplugin(arg);
 
516
        if(p != NULL){
 
517
          p->disabled = true;
 
518
          errno = 0;
 
519
        }
 
520
      }
 
521
      break;
 
522
    case 'e':                   /* --enable */
 
523
      {
 
524
        plugin *p = getplugin(arg);
 
525
        if(p != NULL){
 
526
          p->disabled = false;
 
527
          errno = 0;
 
528
        }
441
529
      }
442
530
      break;
443
531
    case 128:                   /* --plugin-dir */
 
532
      free(plugindir);
444
533
      plugindir = strdup(arg);
445
 
      if(plugindir == NULL){
446
 
        perror("strdup");
447
 
      }      
448
 
      break;
449
 
    case 129:                   /* --config-file */
 
534
      if(plugindir != NULL){
 
535
        errno = 0;
 
536
      }
 
537
      break;
 
538
    case 129:                   /* --config-file */
 
539
      /* This is already done by parse_opt_config_file() */
 
540
      break;
 
541
    case 130:                   /* --userid */
 
542
      tmp_id = strtoimax(arg, &tmp, 10);
 
543
      if(errno != 0 or tmp == arg or *tmp != '\0'
 
544
         or tmp_id != (uid_t)tmp_id){
 
545
        argp_error(state, "Bad user ID number: \"%s\", using %"
 
546
                   PRIdMAX, arg, (intmax_t)uid);
 
547
        break;
 
548
      }
 
549
      uid = (uid_t)tmp_id;
 
550
      errno = 0;
 
551
      break;
 
552
    case 131:                   /* --groupid */
 
553
      tmp_id = strtoimax(arg, &tmp, 10);
 
554
      if(errno != 0 or tmp == arg or *tmp != '\0'
 
555
         or tmp_id != (gid_t)tmp_id){
 
556
        argp_error(state, "Bad group ID number: \"%s\", using %"
 
557
                   PRIdMAX, arg, (intmax_t)gid);
 
558
        break;
 
559
      }
 
560
      gid = (gid_t)tmp_id;
 
561
      errno = 0;
 
562
      break;
 
563
    case 132:                   /* --debug */
 
564
      debug = true;
 
565
      break;
 
566
    case 133:                   /* --plugin-helper-dir */
 
567
      free(pluginhelperdir);
 
568
      pluginhelperdir = strdup(arg);
 
569
      if(pluginhelperdir != NULL){
 
570
        errno = 0;
 
571
      }
 
572
      break;
 
573
      /*
 
574
       * These reproduce what we would get without ARGP_NO_HELP
 
575
       */
 
576
    case '?':                   /* --help */
 
577
      state->flags &= ~(unsigned int)ARGP_NO_EXIT; /* force exit */
 
578
      argp_state_help(state, state->out_stream, ARGP_HELP_STD_HELP);
 
579
      __builtin_unreachable();
 
580
    case -3:                    /* --usage */
 
581
      state->flags &= ~(unsigned int)ARGP_NO_EXIT; /* force exit */
 
582
      argp_state_help(state, state->out_stream,
 
583
                      ARGP_HELP_USAGE | ARGP_HELP_EXIT_OK);
 
584
      __builtin_unreachable();
 
585
    case 'V':                   /* --version */
 
586
      fprintf(state->out_stream, "%s\n", argp_program_version);
 
587
      exit(EXIT_SUCCESS);
 
588
      break;
 
589
/*
 
590
 * When adding more options before this line, remember to also add a
 
591
 * "case" to the "parse_opt_config_file" function below.
 
592
 */
 
593
    case ARGP_KEY_ARG:
 
594
      /* Cryptsetup always passes an argument, which is an empty
 
595
         string if "none" was specified in /etc/crypttab.  So if
 
596
         argument was empty, we ignore it silently. */
 
597
      if(arg[0] == '\0'){
 
598
        break;
 
599
      }
 
600
#if __GNUC__ >= 7
 
601
      __attribute__((fallthrough));
 
602
#else
 
603
          /* FALLTHROUGH */
 
604
#endif
 
605
    default:
 
606
      return ARGP_ERR_UNKNOWN;
 
607
    }
 
608
    return errno;               /* Set to 0 at start */
 
609
  }
 
610
  
 
611
  /* This option parser is the same as parse_opt() above, except it
 
612
     ignores everything but the --config-file option. */
 
613
  error_t parse_opt_config_file(int key, char *arg,
 
614
                                __attribute__((unused))
 
615
                                struct argp_state *state){
 
616
    errno = 0;
 
617
    switch(key){
 
618
    case 'g':                   /* --global-options */
 
619
    case 'G':                   /* --global-env */
 
620
    case 'o':                   /* --options-for */
 
621
    case 'E':                   /* --env-for */
 
622
    case 'd':                   /* --disable */
 
623
    case 'e':                   /* --enable */
 
624
    case 128:                   /* --plugin-dir */
 
625
      break;
 
626
    case 129:                   /* --config-file */
 
627
      free(argfile);
450
628
      argfile = strdup(arg);
451
 
      if(argfile == NULL){
452
 
        perror("strdup");
 
629
      if(argfile != NULL){
 
630
        errno = 0;
453
631
      }
454
 
      break;      
 
632
      break;
455
633
    case 130:                   /* --userid */
456
 
      uid = (uid_t)strtol(arg, NULL, 10);
457
 
      break;
458
634
    case 131:                   /* --groupid */
459
 
      gid = (gid_t)strtol(arg, NULL, 10);
460
 
      break;
461
635
    case 132:                   /* --debug */
462
 
      debug = true;
463
 
      break;
 
636
    case 133:                   /* --plugin-helper-dir */
 
637
    case '?':                   /* --help */
 
638
    case -3:                    /* --usage */
 
639
    case 'V':                   /* --version */
464
640
    case ARGP_KEY_ARG:
465
 
      fprintf(stderr, "Ignoring unknown argument \"%s\"\n", arg);
466
 
      break;
467
 
    case ARGP_KEY_END:
468
641
      break;
469
642
    default:
470
643
      return ARGP_ERR_UNKNOWN;
471
644
    }
472
 
    return 0;
 
645
    return errno;
473
646
  }
474
647
  
475
 
  struct argp argp = { .options = options, .parser = parse_opt,
476
 
                       .args_doc = "[+PLUS_SEPARATED_OPTIONS]",
 
648
  struct argp argp = { .options = options,
 
649
                       .parser = parse_opt_config_file,
 
650
                       .args_doc = "",
477
651
                       .doc = "Mandos plugin runner -- Run plugins" };
478
652
  
479
 
  ret = argp_parse (&argp, argc, argv, 0, 0, NULL);
480
 
  if (ret == ARGP_ERR_UNKNOWN){
481
 
    fprintf(stderr, "Unknown error while parsing arguments\n");
482
 
    exitstatus = EXIT_FAILURE;
 
653
  /* Parse using parse_opt_config_file() in order to get the custom
 
654
     config file location, if any. */
 
655
  ret = argp_parse(&argp, argc, argv,
 
656
                   ARGP_IN_ORDER | ARGP_NO_EXIT | ARGP_NO_HELP,
 
657
                   NULL, NULL);
 
658
  switch(ret){
 
659
  case 0:
 
660
    break;
 
661
  case ENOMEM:
 
662
  default:
 
663
    errno = ret;
 
664
    error(0, errno, "argp_parse");
 
665
    exitstatus = EX_OSERR;
 
666
    goto fallback;
 
667
  case EINVAL:
 
668
    exitstatus = EX_USAGE;
483
669
    goto fallback;
484
670
  }
485
 
 
486
 
  /* Opens the configfile if aviable */
487
 
  if (argfile == NULL){
 
671
  
 
672
  /* Reset to the normal argument parser */
 
673
  argp.parser = parse_opt;
 
674
  
 
675
  /* Open the configfile if available */
 
676
  if(argfile == NULL){
488
677
    conffp = fopen(AFILE, "r");
489
678
  } else {
490
679
    conffp = fopen(argfile, "r");
491
 
  }  
 
680
  }
492
681
  if(conffp != NULL){
493
682
    char *org_line = NULL;
494
683
    char *p, *arg, *new_arg, *line;
495
684
    size_t size = 0;
496
 
    ssize_t sret;
497
685
    const char whitespace_delims[] = " \r\t\f\v\n";
498
686
    const char comment_delim[] = "#";
499
 
 
 
687
    
500
688
    custom_argc = 1;
501
689
    custom_argv = malloc(sizeof(char*) * 2);
502
690
    if(custom_argv == NULL){
503
 
      perror("malloc");
504
 
      exitstatus = EXIT_FAILURE;
 
691
      error(0, errno, "malloc");
 
692
      exitstatus = EX_OSERR;
505
693
      goto fallback;
506
694
    }
507
695
    custom_argv[0] = argv[0];
508
696
    custom_argv[1] = NULL;
509
 
 
510
 
    /* for each line in the config file, strip whitespace and ignore commented text */
 
697
    
 
698
    /* for each line in the config file, strip whitespace and ignore
 
699
       commented text */
511
700
    while(true){
512
701
      sret = getline(&org_line, &size, conffp);
513
702
      if(sret == -1){
514
703
        break;
515
704
      }
516
 
 
 
705
      
517
706
      line = org_line;
518
707
      arg = strsep(&line, comment_delim);
519
708
      while((p = strsep(&arg, whitespace_delims)) != NULL){
522
711
        }
523
712
        new_arg = strdup(p);
524
713
        if(new_arg == NULL){
525
 
          perror("strdup");
526
 
          exitstatus = EXIT_FAILURE;
 
714
          error(0, errno, "strdup");
 
715
          exitstatus = EX_OSERR;
527
716
          free(org_line);
528
717
          goto fallback;
529
718
        }
530
719
        
531
720
        custom_argc += 1;
532
 
        custom_argv = realloc(custom_argv, sizeof(char *)
533
 
                              * ((unsigned int) custom_argc + 1));
534
 
        if(custom_argv == NULL){
535
 
          perror("realloc");
536
 
          exitstatus = EXIT_FAILURE;
537
 
          free(org_line);
538
 
          goto fallback;
 
721
        {
 
722
#if defined(__GLIBC_PREREQ) and __GLIBC_PREREQ(2, 26)
 
723
          char **new_argv = reallocarray(custom_argv, (size_t)custom_argc + 1,
 
724
                                         sizeof(char *));
 
725
#else
 
726
          char **new_argv = NULL;
 
727
          if(((size_t)custom_argc + 1) > (SIZE_MAX / sizeof(char *))){
 
728
            /* overflow */
 
729
            errno = ENOMEM;
 
730
          } else {
 
731
            new_argv = realloc(custom_argv, ((size_t)custom_argc + 1)
 
732
                               * sizeof(char *));
 
733
          }
 
734
#endif
 
735
          if(new_argv == NULL){
 
736
            error(0, errno, "reallocarray");
 
737
            exitstatus = EX_OSERR;
 
738
            free(new_arg);
 
739
            free(org_line);
 
740
            goto fallback;
 
741
          } else {
 
742
            custom_argv = new_argv;
 
743
          }
539
744
        }
540
745
        custom_argv[custom_argc-1] = new_arg;
541
 
        custom_argv[custom_argc] = NULL;        
 
746
        custom_argv[custom_argc] = NULL;
542
747
      }
543
748
    }
 
749
    do {
 
750
      ret = fclose(conffp);
 
751
    } while(ret == EOF and errno == EINTR);
 
752
    if(ret == EOF){
 
753
      error(0, errno, "fclose");
 
754
      exitstatus = EX_IOERR;
 
755
      goto fallback;
 
756
    }
544
757
    free(org_line);
545
 
  } else{
 
758
  } else {
546
759
    /* Check for harmful errors and go to fallback. Other errors might
547
760
       not affect opening plugins */
548
 
    if (errno == EMFILE or errno == ENFILE or errno == ENOMEM){
549
 
      perror("fopen");
550
 
      exitstatus = EXIT_FAILURE;
 
761
    if(errno == EMFILE or errno == ENFILE or errno == ENOMEM){
 
762
      error(0, errno, "fopen");
 
763
      exitstatus = EX_OSERR;
551
764
      goto fallback;
552
765
    }
553
766
  }
554
 
  /* If there was any arguments from configuration file,
555
 
     pass them to parser as command arguments */
 
767
  /* If there were any arguments from the configuration file, pass
 
768
     them to parser as command line arguments */
556
769
  if(custom_argv != NULL){
557
 
    ret = argp_parse (&argp, custom_argc, custom_argv, 0, 0, NULL);
558
 
    if (ret == ARGP_ERR_UNKNOWN){
559
 
      fprintf(stderr, "Unknown error while parsing arguments\n");
560
 
      exitstatus = EXIT_FAILURE;
561
 
      goto fallback;
 
770
    ret = argp_parse(&argp, custom_argc, custom_argv,
 
771
                     ARGP_IN_ORDER | ARGP_NO_EXIT | ARGP_NO_HELP,
 
772
                     NULL, NULL);
 
773
    switch(ret){
 
774
    case 0:
 
775
      break;
 
776
    case ENOMEM:
 
777
    default:
 
778
      errno = ret;
 
779
      error(0, errno, "argp_parse");
 
780
      exitstatus = EX_OSERR;
 
781
      goto fallback;
 
782
    case EINVAL:
 
783
      exitstatus = EX_CONFIG;
 
784
      goto fallback;
 
785
    }
 
786
  }
 
787
  
 
788
  /* Parse actual command line arguments, to let them override the
 
789
     config file */
 
790
  ret = argp_parse(&argp, argc, argv,
 
791
                   ARGP_IN_ORDER | ARGP_NO_EXIT | ARGP_NO_HELP,
 
792
                   NULL, NULL);
 
793
  switch(ret){
 
794
  case 0:
 
795
    break;
 
796
  case ENOMEM:
 
797
  default:
 
798
    errno = ret;
 
799
    error(0, errno, "argp_parse");
 
800
    exitstatus = EX_OSERR;
 
801
    goto fallback;
 
802
  case EINVAL:
 
803
    exitstatus = EX_USAGE;
 
804
    goto fallback;
 
805
  }
 
806
  
 
807
  {
 
808
    char *pluginhelperenv;
 
809
    bool bret = true;
 
810
    ret = asprintf(&pluginhelperenv, "MANDOSPLUGINHELPERDIR=%s",
 
811
                   pluginhelperdir != NULL ? pluginhelperdir : PHDIR);
 
812
    if(ret != -1){
 
813
      bret = add_environment(getplugin(NULL), pluginhelperenv, true);
 
814
    }
 
815
    if(ret == -1 or not bret){
 
816
      error(0, errno, "Failed to set MANDOSPLUGINHELPERDIR"
 
817
            " environment variable to \"%s\" for all plugins\n",
 
818
            pluginhelperdir != NULL ? pluginhelperdir : PHDIR);
 
819
    }
 
820
    if(ret != -1){
 
821
      free(pluginhelperenv);
562
822
    }
563
823
  }
564
824
  
565
825
  if(debug){
566
 
    for(plugin *p = plugin_list; p != NULL; p=p->next){
 
826
    for(plugin *p = plugin_list; p != NULL; p = p->next){
567
827
      fprintf(stderr, "Plugin: %s has %d arguments\n",
568
828
              p->name ? p->name : "Global", p->argc - 1);
569
829
      for(char **a = p->argv; *a != NULL; a++){
570
830
        fprintf(stderr, "\tArg: %s\n", *a);
571
831
      }
572
 
      fprintf(stderr, "...and %u environment variables\n", p->envc);
 
832
      fprintf(stderr, "...and %d environment variables\n", p->envc);
573
833
      for(char **a = p->environ; *a != NULL; a++){
574
834
        fprintf(stderr, "\t%s\n", *a);
575
835
      }
576
836
    }
577
837
  }
578
 
 
579
 
  /* Strip permissions down to nobody */
 
838
  
 
839
  if(getuid() == 0){
 
840
    /* Work around Debian bug #633582:
 
841
       <https://bugs.debian.org/633582> */
 
842
    int plugindir_fd = open(/* plugindir or */ PDIR, O_RDONLY);
 
843
    if(plugindir_fd == -1){
 
844
      if(errno != ENOENT){
 
845
        error(0, errno, "open(\"" PDIR "\")");
 
846
      }
 
847
    } else {
 
848
      ret = (int)TEMP_FAILURE_RETRY(fstat(plugindir_fd, &st));
 
849
      if(ret == -1){
 
850
        error(0, errno, "fstat");
 
851
      } else {
 
852
        if(S_ISDIR(st.st_mode) and st.st_uid == 0 and st.st_gid == 0){
 
853
          ret = fchown(plugindir_fd, uid, gid);
 
854
          if(ret == -1){
 
855
            error(0, errno, "fchown");
 
856
          }
 
857
        }
 
858
      }
 
859
      close(plugindir_fd);
 
860
    }
 
861
  }
 
862
  
 
863
  /* Lower permissions */
 
864
  ret = setgid(gid);
 
865
  if(ret == -1){
 
866
    error(0, errno, "setgid");
 
867
  }
580
868
  ret = setuid(uid);
581
 
  if (ret == -1){
582
 
    perror("setuid");
583
 
  }  
584
 
  setgid(gid);
585
 
  if (ret == -1){
586
 
    perror("setgid");
587
 
  }
588
 
 
589
 
  if (plugindir == NULL){
590
 
    dir = opendir(PDIR);
591
 
  } else {
592
 
    dir = opendir(plugindir);
593
 
  }
594
 
  
595
 
  if(dir == NULL){
596
 
    perror("Could not open plugin dir");
597
 
    exitstatus = EXIT_FAILURE;
598
 
    goto fallback;
599
 
  }
600
 
  
601
 
  /* Set the FD_CLOEXEC flag on the directory, if possible */
 
869
  if(ret == -1){
 
870
    error(0, errno, "setuid");
 
871
  }
 
872
  
 
873
  /* Open plugin directory with close_on_exec flag */
602
874
  {
603
 
    int dir_fd = dirfd(dir);
604
 
    if(dir_fd >= 0){
605
 
      ret = set_cloexec_flag(dir_fd);
606
 
      if(ret < 0){
607
 
        perror("set_cloexec_flag");
608
 
        exitstatus = EXIT_FAILURE;
609
 
        goto fallback;
 
875
    dir_fd = open(plugindir != NULL ? plugindir : PDIR, O_RDONLY |
 
876
#ifdef O_CLOEXEC
 
877
                  O_CLOEXEC
 
878
#else  /* not O_CLOEXEC */
 
879
                  0
 
880
#endif  /* not O_CLOEXEC */
 
881
                  );
 
882
    if(dir_fd == -1){
 
883
      error(0, errno, "Could not open plugin dir");
 
884
      exitstatus = EX_UNAVAILABLE;
 
885
      goto fallback;
 
886
    }
 
887
    
 
888
#ifndef O_CLOEXEC
 
889
  /* Set the FD_CLOEXEC flag on the directory */
 
890
    ret = set_cloexec_flag(dir_fd);
 
891
    if(ret < 0){
 
892
      error(0, errno, "set_cloexec_flag");
 
893
      exitstatus = EX_OSERR;
 
894
      goto fallback;
 
895
    }
 
896
#endif  /* O_CLOEXEC */
 
897
  }
 
898
  
 
899
  int good_name(const struct dirent * const dirent){
 
900
    const char * const patterns[] = { ".*", "#*#", "*~", "*.dpkg-new",
 
901
                                      "*.dpkg-old", "*.dpkg-bak",
 
902
                                      "*.dpkg-divert", NULL };
 
903
#ifdef __GNUC__
 
904
#pragma GCC diagnostic push
 
905
#pragma GCC diagnostic ignored "-Wcast-qual"
 
906
#endif
 
907
    for(const char **pat = (const char **)patterns;
 
908
        *pat != NULL; pat++){
 
909
#ifdef __GNUC__
 
910
#pragma GCC diagnostic pop
 
911
#endif
 
912
      if(fnmatch(*pat, dirent->d_name, FNM_FILE_NAME | FNM_PERIOD)
 
913
         != FNM_NOMATCH){
 
914
        if(debug){
 
915
            fprintf(stderr, "Ignoring plugin dir entry \"%s\""
 
916
                    " matching pattern %s\n", dirent->d_name, *pat);
 
917
        }
 
918
        return 0;
610
919
      }
611
920
    }
 
921
    return 1;
 
922
  }
 
923
  
 
924
  int numplugins = scandirat(dir_fd, ".", &direntries, good_name,
 
925
                             alphasort);
 
926
  if(numplugins == -1){
 
927
    error(0, errno, "Could not scan plugin dir");
 
928
    direntries = NULL;
 
929
    exitstatus = EX_OSERR;
 
930
    goto fallback;
612
931
  }
613
932
  
614
933
  FD_ZERO(&rfds_all);
615
 
 
 
934
  
616
935
  /* Read and execute any executable in the plugin directory*/
617
 
  while(true){
618
 
    dirst = readdir(dir);
619
 
    
620
 
    // All directory entries have been processed
621
 
    if(dirst == NULL){
622
 
      if (errno == EBADF){
623
 
        perror("readdir");
624
 
        exitstatus = EXIT_FAILURE;
625
 
        goto fallback;
626
 
      }
627
 
      break;
628
 
    }
629
 
    
630
 
    d_name_len = strlen(dirst->d_name);
631
 
    
632
 
    // Ignore dotfiles, backup files and other junk
633
 
    {
634
 
      bool bad_name = false;
635
 
      
636
 
      const char const *bad_prefixes[] = { ".", "#", NULL };
637
 
      
638
 
      const char const *bad_suffixes[] = { "~", "#", ".dpkg-new",
639
 
                                           ".dpkg-old",
640
 
                                           ".dpkg-divert", NULL };
641
 
      for(const char **pre = bad_prefixes; *pre != NULL; pre++){
642
 
        size_t pre_len = strlen(*pre);
643
 
        if((d_name_len >= pre_len)
644
 
           and strncmp((dirst->d_name), *pre, pre_len) == 0){
645
 
          if(debug){
646
 
            fprintf(stderr, "Ignoring plugin dir entry \"%s\""
647
 
                    " with bad prefix %s\n", dirst->d_name, *pre);
648
 
          }
649
 
          bad_name = true;
650
 
          break;
651
 
        }
652
 
      }
653
 
      if(bad_name){
654
 
        continue;
655
 
      }
656
 
      for(const char **suf = bad_suffixes; *suf != NULL; suf++){
657
 
        size_t suf_len = strlen(*suf);
658
 
        if((d_name_len >= suf_len)
659
 
           and (strcmp((dirst->d_name)+d_name_len-suf_len, *suf)
660
 
                == 0)){
661
 
          if(debug){
662
 
            fprintf(stderr, "Ignoring plugin dir entry \"%s\""
663
 
                    " with bad suffix %s\n", dirst->d_name, *suf);
664
 
          }
665
 
          bad_name = true;
666
 
          break;
667
 
        }
668
 
      }
669
 
      
670
 
      if(bad_name){
671
 
        continue;
672
 
      }
673
 
    }
674
 
 
675
 
    char *filename;
676
 
    ret = asprintf(&filename, "%s/%s", plugindir, dirst->d_name);
677
 
    if(ret < 0){
678
 
      perror("asprintf");
679
 
      continue;
680
 
    }
681
 
    
682
 
    ret = stat(filename, &st);
683
 
    if (ret == -1){
684
 
      perror("stat");
685
 
      free(filename);
686
 
      continue;
687
 
    }
688
 
 
 
936
  for(int i = 0; i < numplugins; i++){
 
937
    
 
938
    int plugin_fd = openat(dir_fd, direntries[i]->d_name, O_RDONLY);
 
939
    if(plugin_fd == -1){
 
940
      error(0, errno, "Could not open plugin");
 
941
      free(direntries[i]);
 
942
      continue;
 
943
    }
 
944
    ret = (int)TEMP_FAILURE_RETRY(fstat(plugin_fd, &st));
 
945
    if(ret == -1){
 
946
      error(0, errno, "stat");
 
947
      close(plugin_fd);
 
948
      free(direntries[i]);
 
949
      continue;
 
950
    }
 
951
    
689
952
    /* Ignore non-executable files */
690
 
    if (not S_ISREG(st.st_mode) or (access(filename, X_OK) != 0)){
 
953
    if(not S_ISREG(st.st_mode)
 
954
       or (TEMP_FAILURE_RETRY(faccessat(dir_fd, direntries[i]->d_name,
 
955
                                        X_OK, 0)) != 0)){
691
956
      if(debug){
692
 
        fprintf(stderr, "Ignoring plugin dir entry \"%s\""
693
 
                " with bad type or mode\n", filename);
 
957
        fprintf(stderr, "Ignoring plugin dir entry \"%s/%s\""
 
958
                " with bad type or mode\n",
 
959
                plugindir != NULL ? plugindir : PDIR,
 
960
                direntries[i]->d_name);
694
961
      }
695
 
      free(filename);
 
962
      close(plugin_fd);
 
963
      free(direntries[i]);
696
964
      continue;
697
965
    }
698
966
    
699
 
    plugin *p = getplugin(dirst->d_name);
 
967
    plugin *p = getplugin(direntries[i]->d_name);
700
968
    if(p == NULL){
701
 
      perror("getplugin");
702
 
      free(filename);
 
969
      error(0, errno, "getplugin");
 
970
      close(plugin_fd);
 
971
      free(direntries[i]);
703
972
      continue;
704
973
    }
705
974
    if(p->disabled){
706
975
      if(debug){
707
976
        fprintf(stderr, "Ignoring disabled plugin \"%s\"\n",
708
 
                dirst->d_name);
 
977
                direntries[i]->d_name);
709
978
      }
710
 
      free(filename);
 
979
      close(plugin_fd);
 
980
      free(direntries[i]);
711
981
      continue;
712
982
    }
713
983
    {
716
986
      if(g != NULL){
717
987
        for(char **a = g->argv + 1; *a != NULL; a++){
718
988
          if(not add_argument(p, *a)){
719
 
            perror("add_argument");
 
989
            error(0, errno, "add_argument");
720
990
          }
721
991
        }
722
992
        /* Add global environment variables */
723
993
        for(char **e = g->environ; *e != NULL; e++){
724
 
          if(not add_environment(p, *e)){
725
 
            perror("add_environment");
 
994
          if(not add_environment(p, *e, false)){
 
995
            error(0, errno, "add_environment");
726
996
          }
727
997
        }
728
998
      }
729
999
    }
730
 
    /* If this plugin has any environment variables, we will call
731
 
       using execve and need to duplicate the environment from this
732
 
       process, too. */
 
1000
    /* If this plugin has any environment variables, we need to
 
1001
       duplicate the environment from this process, too. */
733
1002
    if(p->environ[0] != NULL){
734
1003
      for(char **e = environ; *e != NULL; e++){
735
 
        char *copy = strdup(*e);
736
 
        if(copy == NULL){
737
 
          perror("strdup");
738
 
          continue;
739
 
        }
740
 
        if(not add_environment(p, copy)){
741
 
          perror("add_environment");
 
1004
        if(not add_environment(p, *e, false)){
 
1005
          error(0, errno, "add_environment");
742
1006
        }
743
1007
      }
744
1008
    }
745
1009
    
746
1010
    int pipefd[2];
747
 
    ret = pipe(pipefd);
748
 
    if (ret == -1){
749
 
      perror("pipe");
750
 
      exitstatus = EXIT_FAILURE;
751
 
      goto fallback;
752
 
    }
 
1011
#ifndef O_CLOEXEC
 
1012
    ret = (int)TEMP_FAILURE_RETRY(pipe(pipefd));
 
1013
#else  /* O_CLOEXEC */
 
1014
    ret = (int)TEMP_FAILURE_RETRY(pipe2(pipefd, O_CLOEXEC));
 
1015
#endif  /* O_CLOEXEC */
 
1016
    if(ret == -1){
 
1017
      error(0, errno, "pipe");
 
1018
      exitstatus = EX_OSERR;
 
1019
      free(direntries[i]);
 
1020
      goto fallback;
 
1021
    }
 
1022
    if(pipefd[0] >= FD_SETSIZE){
 
1023
      fprintf(stderr, "pipe()[0] (%d) >= FD_SETSIZE (%d)", pipefd[0],
 
1024
              FD_SETSIZE);
 
1025
      close(pipefd[0]);
 
1026
      close(pipefd[1]);
 
1027
      exitstatus = EX_OSERR;
 
1028
      free(direntries[i]);
 
1029
      goto fallback;
 
1030
    }
 
1031
#ifndef O_CLOEXEC
753
1032
    /* Ask OS to automatic close the pipe on exec */
754
1033
    ret = set_cloexec_flag(pipefd[0]);
755
1034
    if(ret < 0){
756
 
      perror("set_cloexec_flag");
757
 
      exitstatus = EXIT_FAILURE;
 
1035
      error(0, errno, "set_cloexec_flag");
 
1036
      close(pipefd[0]);
 
1037
      close(pipefd[1]);
 
1038
      exitstatus = EX_OSERR;
 
1039
      free(direntries[i]);
758
1040
      goto fallback;
759
1041
    }
760
1042
    ret = set_cloexec_flag(pipefd[1]);
761
1043
    if(ret < 0){
762
 
      perror("set_cloexec_flag");
763
 
      exitstatus = EXIT_FAILURE;
 
1044
      error(0, errno, "set_cloexec_flag");
 
1045
      close(pipefd[0]);
 
1046
      close(pipefd[1]);
 
1047
      exitstatus = EX_OSERR;
 
1048
      free(direntries[i]);
764
1049
      goto fallback;
765
1050
    }
 
1051
#endif  /* not O_CLOEXEC */
766
1052
    /* Block SIGCHLD until process is safely in process list */
767
 
    ret = sigprocmask (SIG_BLOCK, &sigchld_action.sa_mask, NULL);
 
1053
    ret = (int)TEMP_FAILURE_RETRY(sigprocmask(SIG_BLOCK,
 
1054
                                              &sigchld_action.sa_mask,
 
1055
                                              NULL));
768
1056
    if(ret < 0){
769
 
      perror("sigprocmask");
770
 
      exitstatus = EXIT_FAILURE;
 
1057
      error(0, errno, "sigprocmask");
 
1058
      exitstatus = EX_OSERR;
 
1059
      free(direntries[i]);
771
1060
      goto fallback;
772
1061
    }
773
 
    // Starting a new process to be watched
774
 
    pid_t pid = fork();
 
1062
    /* Starting a new process to be watched */
 
1063
    pid_t pid;
 
1064
    do {
 
1065
      pid = fork();
 
1066
    } while(pid == -1 and errno == EINTR);
775
1067
    if(pid == -1){
776
 
      perror("fork");
777
 
      exitstatus = EXIT_FAILURE;
 
1068
      error(0, errno, "fork");
 
1069
      TEMP_FAILURE_RETRY(sigprocmask(SIG_UNBLOCK,
 
1070
                                     &sigchld_action.sa_mask, NULL));
 
1071
      close(pipefd[0]);
 
1072
      close(pipefd[1]);
 
1073
      exitstatus = EX_OSERR;
 
1074
      free(direntries[i]);
778
1075
      goto fallback;
779
1076
    }
780
1077
    if(pid == 0){
781
1078
      /* this is the child process */
782
1079
      ret = sigaction(SIGCHLD, &old_sigchld_action, NULL);
783
1080
      if(ret < 0){
784
 
        perror("sigaction");
785
 
        _exit(EXIT_FAILURE);
 
1081
        error(0, errno, "sigaction");
 
1082
        _exit(EX_OSERR);
786
1083
      }
787
 
      ret = sigprocmask (SIG_UNBLOCK, &sigchld_action.sa_mask, NULL);
 
1084
      ret = sigprocmask(SIG_UNBLOCK, &sigchld_action.sa_mask, NULL);
788
1085
      if(ret < 0){
789
 
        perror("sigprocmask");
790
 
        _exit(EXIT_FAILURE);
 
1086
        error(0, errno, "sigprocmask");
 
1087
        _exit(EX_OSERR);
791
1088
      }
792
 
 
 
1089
      
793
1090
      ret = dup2(pipefd[1], STDOUT_FILENO); /* replace our stdout */
794
1091
      if(ret == -1){
795
 
        perror("dup2");
796
 
        _exit(EXIT_FAILURE);
 
1092
        error(0, errno, "dup2");
 
1093
        _exit(EX_OSERR);
797
1094
      }
798
1095
      
799
 
      if(dirfd(dir) < 0){
800
 
        /* If dir has no file descriptor, we could not set FD_CLOEXEC
801
 
           above and must now close it manually here. */
802
 
        closedir(dir);
803
 
      }
804
 
      if(p->environ[0] == NULL){
805
 
        if(execv(filename, p->argv) < 0){
806
 
          perror("execv");
807
 
          _exit(EXIT_FAILURE);
808
 
        }
809
 
      } else {
810
 
        if(execve(filename, p->argv, p->environ) < 0){
811
 
          perror("execve");
812
 
          _exit(EXIT_FAILURE);
813
 
        }
 
1096
      if(fexecve(plugin_fd, p->argv,
 
1097
                (p->environ[0] != NULL) ? p->environ : environ) < 0){
 
1098
        error(0, errno, "fexecve for %s/%s",
 
1099
              plugindir != NULL ? plugindir : PDIR,
 
1100
              direntries[i]->d_name);
 
1101
        _exit(EX_OSERR);
814
1102
      }
815
1103
      /* no return */
816
1104
    }
817
1105
    /* Parent process */
818
1106
    close(pipefd[1]);           /* Close unused write end of pipe */
819
 
    free(filename);
820
 
    plugin *new_plugin = getplugin(dirst->d_name);
821
 
    if (new_plugin == NULL){
822
 
      perror("getplugin");
823
 
      ret = sigprocmask (SIG_UNBLOCK, &sigchld_action.sa_mask, NULL);
 
1107
    close(plugin_fd);
 
1108
    plugin *new_plugin = getplugin(direntries[i]->d_name);
 
1109
    if(new_plugin == NULL){
 
1110
      error(0, errno, "getplugin");
 
1111
      ret = (int)(TEMP_FAILURE_RETRY
 
1112
                  (sigprocmask(SIG_UNBLOCK, &sigchld_action.sa_mask,
 
1113
                               NULL)));
824
1114
      if(ret < 0){
825
 
        perror("sigprocmask");
 
1115
        error(0, errno, "sigprocmask");
826
1116
      }
827
 
      exitstatus = EXIT_FAILURE;
 
1117
      exitstatus = EX_OSERR;
 
1118
      free(direntries[i]);
828
1119
      goto fallback;
829
1120
    }
 
1121
    free(direntries[i]);
830
1122
    
831
1123
    new_plugin->pid = pid;
832
1124
    new_plugin->fd = pipefd[0];
833
 
    
 
1125
 
 
1126
    if(debug){
 
1127
      fprintf(stderr, "Plugin %s started (PID %" PRIdMAX ")\n",
 
1128
              new_plugin->name, (intmax_t) (new_plugin->pid));
 
1129
    }
 
1130
 
834
1131
    /* Unblock SIGCHLD so signal handler can be run if this process
835
1132
       has already completed */
836
 
    ret = sigprocmask (SIG_UNBLOCK, &sigchld_action.sa_mask, NULL);
 
1133
    ret = (int)TEMP_FAILURE_RETRY(sigprocmask(SIG_UNBLOCK,
 
1134
                                              &sigchld_action.sa_mask,
 
1135
                                              NULL));
837
1136
    if(ret < 0){
838
 
      perror("sigprocmask");
839
 
      exitstatus = EXIT_FAILURE;
 
1137
      error(0, errno, "sigprocmask");
 
1138
      exitstatus = EX_OSERR;
840
1139
      goto fallback;
841
1140
    }
842
1141
    
843
1142
    FD_SET(new_plugin->fd, &rfds_all);
844
1143
    
845
 
    if (maxfd < new_plugin->fd){
 
1144
    if(maxfd < new_plugin->fd){
846
1145
      maxfd = new_plugin->fd;
847
1146
    }
848
 
    
849
1147
  }
850
1148
  
851
 
  closedir(dir);
852
 
  dir = NULL;
853
 
 
 
1149
  free(direntries);
 
1150
  direntries = NULL;
 
1151
  close(dir_fd);
 
1152
  dir_fd = -1;
 
1153
  free_plugin(getplugin(NULL));
 
1154
  
854
1155
  for(plugin *p = plugin_list; p != NULL; p = p->next){
855
1156
    if(p->pid != 0){
856
1157
      break;
861
1162
      free_plugin_list();
862
1163
    }
863
1164
  }
864
 
 
 
1165
  
865
1166
  /* Main loop while running plugins exist */
866
1167
  while(plugin_list){
867
1168
    fd_set rfds = rfds_all;
868
1169
    int select_ret = select(maxfd+1, &rfds, NULL, NULL, NULL);
869
 
    if (select_ret == -1){
870
 
      perror("select");
871
 
      exitstatus = EXIT_FAILURE;
 
1170
    if(select_ret == -1 and errno != EINTR){
 
1171
      error(0, errno, "select");
 
1172
      exitstatus = EX_OSERR;
872
1173
      goto fallback;
873
1174
    }
874
1175
    /* OK, now either a process completed, or something can be read
875
1176
       from one of them */
876
 
    for(plugin *proc = plugin_list; proc != NULL; proc = proc->next){
 
1177
    for(plugin *proc = plugin_list; proc != NULL;){
877
1178
      /* Is this process completely done? */
878
 
      if(proc->eof and proc->completed){
 
1179
      if(proc->completed and proc->eof){
879
1180
        /* Only accept the plugin output if it exited cleanly */
880
1181
        if(not WIFEXITED(proc->status)
881
1182
           or WEXITSTATUS(proc->status) != 0){
882
1183
          /* Bad exit by plugin */
883
 
 
 
1184
          
884
1185
          if(debug){
885
1186
            if(WIFEXITED(proc->status)){
886
 
              fprintf(stderr, "Plugin %u exited with status %d\n",
887
 
                      (unsigned int) (proc->pid),
 
1187
              fprintf(stderr, "Plugin %s [%" PRIdMAX "] exited with"
 
1188
                      " status %d\n", proc->name,
 
1189
                      (intmax_t) (proc->pid),
888
1190
                      WEXITSTATUS(proc->status));
889
 
            } else if(WIFSIGNALED(proc->status)) {
890
 
              fprintf(stderr, "Plugin %u killed by signal %d\n",
891
 
                      (unsigned int) (proc->pid),
892
 
                      WTERMSIG(proc->status));
893
 
            } else if(WCOREDUMP(proc->status)){
894
 
              fprintf(stderr, "Plugin %d dumped core\n",
895
 
                      (unsigned int) (proc->pid));
 
1191
            } else if(WIFSIGNALED(proc->status)){
 
1192
              fprintf(stderr, "Plugin %s [%" PRIdMAX "] killed by"
 
1193
                      " signal %d: %s\n", proc->name,
 
1194
                      (intmax_t) (proc->pid),
 
1195
                      WTERMSIG(proc->status),
 
1196
                      strsignal(WTERMSIG(proc->status)));
896
1197
            }
897
1198
          }
898
1199
          
899
1200
          /* Remove the plugin */
900
1201
          FD_CLR(proc->fd, &rfds_all);
901
 
 
 
1202
          
902
1203
          /* Block signal while modifying process_list */
903
 
          ret = sigprocmask(SIG_BLOCK, &sigchld_action.sa_mask, NULL);
 
1204
          ret = (int)TEMP_FAILURE_RETRY(sigprocmask
 
1205
                                        (SIG_BLOCK,
 
1206
                                         &sigchld_action.sa_mask,
 
1207
                                         NULL));
904
1208
          if(ret < 0){
905
 
            perror("sigprocmask");
906
 
            exitstatus = EXIT_FAILURE;
 
1209
            error(0, errno, "sigprocmask");
 
1210
            exitstatus = EX_OSERR;
907
1211
            goto fallback;
908
1212
          }
 
1213
          
 
1214
          plugin *next_plugin = proc->next;
909
1215
          free_plugin(proc);
 
1216
          proc = next_plugin;
 
1217
          
910
1218
          /* We are done modifying process list, so unblock signal */
911
 
          ret = sigprocmask (SIG_UNBLOCK, &sigchld_action.sa_mask,
912
 
                             NULL);
 
1219
          ret = (int)(TEMP_FAILURE_RETRY
 
1220
                      (sigprocmask(SIG_UNBLOCK,
 
1221
                                   &sigchld_action.sa_mask, NULL)));
913
1222
          if(ret < 0){
914
 
            perror("sigprocmask");
915
 
            exitstatus = EXIT_FAILURE;
 
1223
            error(0, errno, "sigprocmask");
 
1224
            exitstatus = EX_OSERR;
916
1225
            goto fallback;
917
1226
          }
918
1227
          
919
1228
          if(plugin_list == NULL){
920
1229
            break;
921
1230
          }
 
1231
          
922
1232
          continue;
923
1233
        }
924
1234
        
925
1235
        /* This process exited nicely, so print its buffer */
926
 
 
 
1236
        
927
1237
        bool bret = print_out_password(proc->buffer,
928
1238
                                       proc->buffer_length);
929
1239
        if(not bret){
930
 
          perror("print_out_password");
931
 
          exitstatus = EXIT_FAILURE;
 
1240
          error(0, errno, "print_out_password");
 
1241
          exitstatus = EX_IOERR;
932
1242
        }
933
1243
        goto fallback;
934
1244
      }
936
1246
      /* This process has not completed.  Does it have any output? */
937
1247
      if(proc->eof or not FD_ISSET(proc->fd, &rfds)){
938
1248
        /* This process had nothing to say at this time */
 
1249
        proc = proc->next;
939
1250
        continue;
940
1251
      }
941
1252
      /* Before reading, make the process' data buffer large enough */
942
1253
      if(proc->buffer_length + BUFFER_SIZE > proc->buffer_size){
943
 
        proc->buffer = realloc(proc->buffer, proc->buffer_size
944
 
                               + (size_t) BUFFER_SIZE);
945
 
        if (proc->buffer == NULL){
946
 
          perror("malloc");
947
 
          exitstatus = EXIT_FAILURE;
 
1254
        char *new_buffer = realloc(proc->buffer, proc->buffer_size
 
1255
                                   + (size_t) BUFFER_SIZE);
 
1256
        if(new_buffer == NULL){
 
1257
          error(0, errno, "malloc");
 
1258
          exitstatus = EX_OSERR;
948
1259
          goto fallback;
949
1260
        }
 
1261
        proc->buffer = new_buffer;
950
1262
        proc->buffer_size += BUFFER_SIZE;
951
1263
      }
952
1264
      /* Read from the process */
953
 
      ret = read(proc->fd, proc->buffer + proc->buffer_length,
954
 
                 BUFFER_SIZE);
955
 
      if(ret < 0){
 
1265
      sret = TEMP_FAILURE_RETRY(read(proc->fd,
 
1266
                                     proc->buffer
 
1267
                                     + proc->buffer_length,
 
1268
                                     BUFFER_SIZE));
 
1269
      if(sret < 0){
956
1270
        /* Read error from this process; ignore the error */
 
1271
        proc = proc->next;
957
1272
        continue;
958
1273
      }
959
 
      if(ret == 0){
 
1274
      if(sret == 0){
960
1275
        /* got EOF */
961
1276
        proc->eof = true;
962
1277
      } else {
963
 
        proc->buffer_length += (size_t) ret;
 
1278
        proc->buffer_length += (size_t) sret;
964
1279
      }
965
1280
    }
966
1281
  }
967
 
 
968
 
 
 
1282
  
 
1283
  
969
1284
 fallback:
970
1285
  
971
 
  if(plugin_list == NULL or exitstatus != EXIT_SUCCESS){
 
1286
  if(plugin_list == NULL or (exitstatus != EXIT_SUCCESS
 
1287
                             and exitstatus != EX_OK)){
972
1288
    /* Fallback if all plugins failed, none are found or an error
973
1289
       occured */
974
1290
    bool bret;
975
1291
    fprintf(stderr, "Going to fallback mode using getpass(3)\n");
976
1292
    char *passwordbuffer = getpass("Password: ");
977
 
    bret = print_out_password(passwordbuffer, strlen(passwordbuffer));
 
1293
    size_t len = strlen(passwordbuffer);
 
1294
    /* Strip trailing newline */
 
1295
    if(len > 0 and passwordbuffer[len-1] == '\n'){
 
1296
      passwordbuffer[len-1] = '\0'; /* not strictly necessary */
 
1297
      len--;
 
1298
    }
 
1299
    bret = print_out_password(passwordbuffer, len);
978
1300
    if(not bret){
979
 
      perror("print_out_password");
980
 
      exitstatus = EXIT_FAILURE;
 
1301
      error(0, errno, "print_out_password");
 
1302
      exitstatus = EX_IOERR;
981
1303
    }
982
1304
  }
983
1305
  
984
1306
  /* Restore old signal handler */
985
1307
  ret = sigaction(SIGCHLD, &old_sigchld_action, NULL);
986
1308
  if(ret == -1){
987
 
    perror("sigaction");
988
 
    exitstatus = EXIT_FAILURE;
 
1309
    error(0, errno, "sigaction");
 
1310
    exitstatus = EX_OSERR;
989
1311
  }
990
 
 
 
1312
  
991
1313
  if(custom_argv != NULL){
992
1314
    for(char **arg = custom_argv+1; *arg != NULL; arg++){
993
1315
      free(*arg);
995
1317
    free(custom_argv);
996
1318
  }
997
1319
  
998
 
  if(dir != NULL){
999
 
    closedir(dir);
 
1320
  free(direntries);
 
1321
  
 
1322
  if(dir_fd != -1){
 
1323
    close(dir_fd);
1000
1324
  }
1001
1325
  
1002
 
  /* Free the process list and kill the processes */
 
1326
  /* Kill the processes */
1003
1327
  for(plugin *p = plugin_list; p != NULL; p = p->next){
1004
1328
    if(p->pid != 0){
1005
1329
      close(p->fd);
1006
1330
      ret = kill(p->pid, SIGTERM);
1007
1331
      if(ret == -1 and errno != ESRCH){
1008
1332
        /* Set-uid proccesses might not get closed */
1009
 
        perror("kill");
 
1333
        error(0, errno, "kill");
1010
1334
      }
1011
1335
    }
1012
1336
  }
1013
1337
  
1014
1338
  /* Wait for any remaining child processes to terminate */
1015
 
  do{
 
1339
  do {
1016
1340
    ret = wait(NULL);
1017
1341
  } while(ret >= 0);
1018
1342
  if(errno != ECHILD){
1019
 
    perror("wait");
 
1343
    error(0, errno, "wait");
1020
1344
  }
1021
 
 
 
1345
  
1022
1346
  free_plugin_list();
1023
1347
  
1024
1348
  free(plugindir);
 
1349
  free(pluginhelperdir);
1025
1350
  free(argfile);
1026
1351
  
1027
1352
  return exitstatus;