/mandos/release

To get this branch, use:
bzr branch http://bzr.recompile.se/loggerhead/mandos/release

« back to all changes in this revision

Viewing changes to mandos-client.c

  • Committer: Björn Påhlsson
  • Date: 2008-08-12 16:29:05 UTC
  • mto: (237.7.1 mandos) (24.1.154 mandos)
  • mto: This revision was merged to the branch mainline in revision 69.
  • Revision ID: belorn@braxen-20080812162905-6zm7vdvj81wkumwb
mandos-client
        Better error handling for failing to start plugins
        Handle fork errors
password-request
        correct exit routine

Show diffs side-by-side

added added

removed removed

Lines of Context:
mandos-client.c
21
21
 * Contact the authors at <mandos@fukt.bsnet.se>.
22
22
 */
23
23
 
24
 
#define _GNU_SOURCE             /* TEMP_FAILURE_RETRY(), getline(),
25
 
                                   asprintf() */
 
24
#define _GNU_SOURCE             /* TEMP_FAILURE_RETRY() */
 
25
 
26
26
#include <stddef.h>             /* size_t, NULL */
27
27
#include <stdlib.h>             /* malloc(), exit(), EXIT_FAILURE,
28
28
                                   EXIT_SUCCESS, realloc() */
51
51
                                   close() */
52
52
#include <fcntl.h>              /* fcntl(), F_GETFD, F_SETFD,
53
53
                                   FD_CLOEXEC */
54
 
#include <string.h>             /* strsep, strlen(), asprintf() */
 
54
#include <string.h>             /* strtok, strlen(), strcpy(),
 
55
                                   strcat() */
55
56
#include <errno.h>              /* errno */
56
57
#include <argp.h>               /* struct argp_option, struct
57
58
                                   argp_state, struct argp,
64
65
#include <errno.h>              /* errno, EBADF */
65
66
 
66
67
#define BUFFER_SIZE 256
67
 
#define ARGFILE "/conf/conf.d/mandos/plugin-runner.conf"
68
68
 
69
 
const char *argp_program_version = "plugin-runner 1.0";
 
69
const char *argp_program_version = "mandos-client 1.0";
70
70
const char *argp_program_bug_address = "<mandos@fukt.bsnet.se>";
71
71
 
72
72
struct process;
87
87
  char *name;                   /* can be NULL or any plugin name */
88
88
  char **argv;
89
89
  int argc;
90
 
  char **environ;
91
 
  int envc;
92
90
  bool disabled;
93
91
  struct plugin *next;
94
92
} plugin;
103
101
  /* Create a new plugin */
104
102
  plugin *new_plugin = malloc(sizeof(plugin));
105
103
  if (new_plugin == NULL){
106
 
    return NULL;
107
 
  }
108
 
  char *copy_name = NULL;
109
 
  if(name != NULL){
110
 
    copy_name = strdup(name);
111
 
  }
112
 
  if(copy_name == NULL){
113
 
    return NULL;
114
 
  }
115
 
  
116
 
  *new_plugin = (plugin) { .name = copy_name,
117
 
                           .argc = 1,
118
 
                           .envc = 0,
119
 
                           .disabled = false,
120
 
                           .next = *plugin_list };
121
 
  
 
104
    perror("malloc");
 
105
    exit(EXIT_FAILURE);
 
106
  }
 
107
  new_plugin->name = name;
122
108
  new_plugin->argv = malloc(sizeof(char *) * 2);
123
109
  if (new_plugin->argv == NULL){
124
 
    free(copy_name);
125
 
    free(new_plugin);
126
 
    return NULL;
 
110
    perror("malloc");
 
111
    exit(EXIT_FAILURE);
127
112
  }
128
 
  new_plugin->argv[0] = copy_name;
 
113
  new_plugin->argv[0] = name;
129
114
  new_plugin->argv[1] = NULL;
130
 
 
131
 
  new_plugin->environ = malloc(sizeof(char *));
132
 
  if(new_plugin->environ == NULL){
133
 
    free(copy_name);
134
 
    free(new_plugin->argv);
135
 
    free(new_plugin);
136
 
    return NULL;
137
 
  }
138
 
  new_plugin->environ[0] = NULL;
 
115
  new_plugin->argc = 1;
 
116
  new_plugin->disabled = false;
 
117
  new_plugin->next = *plugin_list;
139
118
  /* Append the new plugin to the list */
140
119
  *plugin_list = new_plugin;
141
120
  return new_plugin;
142
121
}
143
122
 
144
 
/* Helper function for add_argument and add_environment */
145
 
static bool add_to_char_array(const char *new, char ***array,
146
 
                              int *len){
147
 
  /* Resize the pointed-to array to hold one more pointer */
148
 
  *array = realloc(*array, sizeof(char *)
149
 
                   * (size_t) ((*len) + 2));
150
 
  /* Malloc check */
151
 
  if(*array == NULL){
152
 
    return false;
153
 
  }
154
 
  /* Make a copy of the new string */
155
 
  char *copy = strdup(new);
156
 
  if(copy == NULL){
157
 
    return false;
158
 
  }
159
 
  /* Insert the copy */
160
 
  (*array)[*len] = copy;
161
 
  (*len)++;
162
 
  /* Add a new terminating NULL pointer to the last element */
163
 
  (*array)[*len] = NULL;
164
 
  return true;
165
 
}
166
 
 
167
 
/* Add to a plugin's argument vector */
168
 
static bool add_argument(plugin *p, const char *arg){
169
 
  if(p == NULL){
170
 
    return false;
171
 
  }
172
 
  return add_to_char_array(arg, &(p->argv), &(p->argc));
173
 
}
174
 
 
175
 
/* Add to a plugin's environment */
176
 
static bool add_environment(plugin *p, const char *def){
177
 
  if(p == NULL){
178
 
    return false;
179
 
  }
180
 
  return add_to_char_array(def, &(p->environ), &(p->envc));
181
 
}
182
 
 
 
123
static void addargument(plugin *p, char *arg){
 
124
  p->argv[p->argc] = arg;
 
125
  p->argv = realloc(p->argv, sizeof(char *) * (size_t)(p->argc + 2));
 
126
  if (p->argv == NULL){
 
127
    perror("malloc");
 
128
    exit(EXIT_FAILURE);
 
129
  }
 
130
  p->argc++;
 
131
  p->argv[p->argc] = NULL;
 
132
}
183
133
 
184
134
/*
185
135
 * Based on the example in the GNU LibC manual chapter 13.13 "File
235
185
  return true;
236
186
}
237
187
 
238
 
char **add_to_argv(char **argv, int *argc, char *arg){
239
 
  if (argv == NULL){
240
 
    *argc = 1;
241
 
    argv = malloc(sizeof(char*) * 2);
242
 
    if(argv == NULL){
243
 
      return NULL;
244
 
    }
245
 
    argv[0] = NULL;     /* Will be set to argv[0] in main before parsing */
246
 
    argv[1] = NULL;
247
 
  }
248
 
  *argc += 1;
249
 
  argv = realloc(argv, sizeof(char *)
250
 
                  * ((unsigned int) *argc + 1));
251
 
  if(argv == NULL){
252
 
    return NULL;
253
 
  }
254
 
  argv[*argc-1] = arg;
255
 
  argv[*argc] = NULL;   
256
 
  return argv;
257
 
}
258
 
 
259
 
static void free_plugin_list(plugin *plugin_list){
260
 
  for(plugin *next = plugin_list; plugin_list != NULL; plugin_list = next){
261
 
    next = plugin_list->next;
262
 
    free(plugin_list->name);
263
 
    for(char **arg = plugin_list->argv; *arg != NULL; arg++){
264
 
      free(*arg);
265
 
    }    
266
 
    free(plugin_list->argv);
267
 
    for(char **env = plugin_list->environ; *env != NULL; env++){
268
 
      free(*env);
269
 
    }
270
 
    free(plugin_list->environ);
271
 
    free(plugin_list);
272
 
  }  
273
 
}
274
 
 
275
188
int main(int argc, char *argv[]){
276
 
  const char *plugindir = "/lib/mandos/plugins.d";
277
 
  const char *argfile = ARGFILE;
278
 
  FILE *conffp;
 
189
  const char *plugindir = "/conf/conf.d/mandos/plugins.d";
279
190
  size_t d_name_len;
280
191
  DIR *dir = NULL;
281
192
  struct dirent *dirst;
289
200
  struct sigaction old_sigchld_action;
290
201
  struct sigaction sigchld_action = { .sa_handler = handle_sigchld,
291
202
                                      .sa_flags = SA_NOCLDSTOP };
292
 
  char **custom_argv = NULL;
293
 
  int custom_argc = 0;
 
203
  char *plus_options = NULL;
 
204
  char **plus_argv = NULL;
 
205
 
 
206
  errno = 0;
294
207
  
295
208
  /* Establish a signal handler */
296
209
  sigemptyset(&sigchld_action.sa_mask);
297
210
  ret = sigaddset(&sigchld_action.sa_mask, SIGCHLD);
298
 
  if(ret == -1){
 
211
  if(ret < 0){
299
212
    perror("sigaddset");
300
 
    exitstatus = EXIT_FAILURE;
301
 
    goto fallback;
 
213
    exit(EXIT_FAILURE);
302
214
  }
303
215
  ret = sigaction(SIGCHLD, &sigchld_action, &old_sigchld_action);
304
 
  if(ret == -1){
 
216
  if(ret < 0){
305
217
    perror("sigaction");
306
 
    exitstatus = EXIT_FAILURE;    
307
 
    goto fallback;
 
218
    exit(EXIT_FAILURE);
308
219
  }
309
220
  
310
221
  /* The options we understand. */
312
223
    { .name = "global-options", .key = 'g',
313
224
      .arg = "OPTION[,OPTION[,...]]",
314
225
      .doc = "Options passed to all plugins" },
315
 
    { .name = "global-envs", .key = 'e',
316
 
      .arg = "VAR=value",
317
 
      .doc = "Environment variable passed to all plugins" },
318
226
    { .name = "options-for", .key = 'o',
319
227
      .arg = "PLUGIN:OPTION[,OPTION[,...]]",
320
228
      .doc = "Options passed only to specified plugin" },
321
 
    { .name = "envs-for", .key = 'f',
322
 
      .arg = "PLUGIN:ENV=value",
323
 
      .doc = "Environment variable passed to specified plugin" },
324
229
    { .name = "disable", .key = 'd',
325
230
      .arg = "PLUGIN",
326
231
      .doc = "Disable a specific plugin", .group = 1 },
345
250
    switch (key) {
346
251
    case 'g':
347
252
      if (arg != NULL){
348
 
        char *p;
349
 
        while((p = strsep(&arg, ",")) != NULL){
350
 
          if(p[0] == '\0'){
351
 
            continue;
352
 
          }
353
 
          if(not add_argument(getplugin(NULL, plugins), p)){
354
 
            perror("add_argument");
355
 
            return ARGP_ERR_UNKNOWN;
356
 
          }
357
 
        }
358
 
      }
359
 
      break;
360
 
    case 'e':
361
 
      if(arg == NULL){
362
 
        break;
363
 
      }
364
 
      {
365
 
        char *envdef = strdup(arg);
366
 
        if(envdef == NULL){
367
 
          break;
368
 
        }
369
 
        if(not add_environment(getplugin(NULL, plugins), envdef)){
370
 
          perror("add_environment");
371
 
        }
 
253
        char *p = strtok(arg, ",");
 
254
        do{
 
255
          addargument(getplugin(NULL, plugins), p);
 
256
          p = strtok(NULL, ",");
 
257
        } while (p != NULL);
372
258
      }
373
259
      break;
374
260
    case 'o':
375
261
      if (arg != NULL){
376
 
        char *p_name = strsep(&arg, ":");
377
 
        if(p_name[0] == '\0'){
378
 
          break;
379
 
        }
380
 
        char *opt = strsep(&arg, ":");
381
 
        if(opt[0] == '\0'){
382
 
          break;
383
 
        }
384
 
        if(opt != NULL){
385
 
          char *p;
386
 
          while((p = strsep(&opt, ",")) != NULL){
387
 
            if(p[0] == '\0'){
388
 
              continue;
389
 
            }
390
 
            if(not add_argument(getplugin(p_name, plugins), p)){
391
 
              perror("add_argument");
392
 
              return ARGP_ERR_UNKNOWN;
393
 
            }
394
 
          }
395
 
        }
396
 
      }
397
 
      break;
398
 
    case 'f':
399
 
      if(arg == NULL){
400
 
        break;
401
 
      }
402
 
      {
403
 
        char *envdef = strchr(arg, ':');
404
 
        if(envdef == NULL){
405
 
          break;
406
 
        }
407
 
        char *p_name = strndup(arg, (size_t) (envdef-arg));
408
 
        if(p_name == NULL){
409
 
          break;
410
 
        }
411
 
        envdef++;
412
 
        if(not add_environment(getplugin(p_name, plugins), envdef)){
413
 
          perror("add_environment");
 
262
        char *name = strtok(arg, ":");
 
263
        char *p = strtok(NULL, ":");
 
264
        if(p != NULL){
 
265
          p = strtok(p, ",");
 
266
          do{
 
267
            addargument(getplugin(name, plugins), p);
 
268
            p = strtok(NULL, ",");
 
269
          } while (p != NULL);
414
270
        }
415
271
      }
416
272
      break;
417
273
    case 'd':
418
274
      if (arg != NULL){
419
 
        plugin *p = getplugin(arg, plugins);
420
 
        if(p == NULL){
421
 
          return ARGP_ERR_UNKNOWN;
422
 
        }
423
 
        p->disabled = true;
 
275
        getplugin(arg, plugins)->disabled = true;
424
276
      }
425
277
      break;
426
278
    case 128:
436
288
      debug = true;
437
289
      break;
438
290
    case ARGP_KEY_ARG:
439
 
      fprintf(stderr, "Ignoring unknown argument \"%s\"\n", arg);
 
291
      if(plus_options != NULL or arg == NULL or arg[0] != '+'){
 
292
        argp_usage (state);
 
293
      }
 
294
      plus_options = arg;
440
295
      break;
441
296
    case ARGP_KEY_END:
442
297
      break;
456
311
  if (ret == ARGP_ERR_UNKNOWN){
457
312
    fprintf(stderr, "Unknown error while parsing arguments\n");
458
313
    exitstatus = EXIT_FAILURE;
459
 
    goto fallback;
 
314
    goto end;
460
315
  }
461
 
 
462
 
  conffp = fopen(argfile, "r");
463
 
  if(conffp != NULL){
464
 
    char *org_line = NULL;
465
 
    char *p, *arg, *new_arg, *line;
466
 
    size_t size = 0;
467
 
    ssize_t sret;
468
 
    const char whitespace_delims[] = " \r\t\f\v\n";
469
 
    const char comment_delim[] = "#";
470
 
 
471
 
    while(true){
472
 
      sret = getline(&org_line, &size, conffp);
473
 
      if(sret == -1){
474
 
        break;
475
 
      }
476
 
 
477
 
      line = org_line;
478
 
      arg = strsep(&line, comment_delim);
479
 
      while((p = strsep(&arg, whitespace_delims)) != NULL){
480
 
        if(p[0] == '\0'){
481
 
          continue;
482
 
        }
483
 
        new_arg = strdup(p);
484
 
        custom_argv = add_to_argv(custom_argv, &custom_argc, new_arg);
485
 
        if (custom_argv == NULL){
486
 
          perror("add_to_argv");
487
 
          exitstatus = EXIT_FAILURE;
488
 
          goto fallback;
489
 
        }
490
 
      }
491
 
    }
492
 
    free(org_line);
493
 
  } else{
494
 
    /* Check for harmful errors and go to fallback. Other errors might
495
 
       not affect opening plugins */
496
 
    if (errno == EMFILE or errno == ENFILE or errno == ENOMEM){
497
 
      perror("fopen");
 
316
  
 
317
  if(plus_options){
 
318
    /* This is a mangled argument in the form of
 
319
     "+--option+--other-option=parameter+--yet-another-option", etc */
 
320
    /* Make new argc and argv vars, and call argp_parse() again. */
 
321
    plus_options++;             /* skip the first '+' character */
 
322
    const char delims[] = "+";
 
323
    char *arg;
 
324
    int new_argc = 1;
 
325
    plus_argv = malloc(sizeof(char*) * 2);
 
326
    if(plus_argv == NULL){
 
327
      perror("malloc");
498
328
      exitstatus = EXIT_FAILURE;
499
 
      goto fallback;
500
 
    }
501
 
  }
502
 
 
503
 
  if(custom_argv != NULL){
504
 
    custom_argv[0] = argv[0];
505
 
    ret = argp_parse (&argp, custom_argc, custom_argv, 0, 0, &plugin_list);
 
329
      goto end;
 
330
    }
 
331
    plus_argv[0] = argv[0];
 
332
    plus_argv[1] = NULL;
 
333
    arg = strtok(plus_options, delims); /* Get first argument */
 
334
    while(arg != NULL){
 
335
      new_argc++;
 
336
      plus_argv = realloc(plus_argv, sizeof(char *)
 
337
                         * ((unsigned int) new_argc + 1));
 
338
      if(plus_argv == NULL){
 
339
        perror("realloc");
 
340
        exitstatus = EXIT_FAILURE;
 
341
        goto end;
 
342
      }
 
343
      plus_argv[new_argc-1] = arg;
 
344
      plus_argv[new_argc] = NULL;
 
345
      arg = strtok(NULL, delims); /* Get next argument */
 
346
    }
 
347
    ret = argp_parse (&argp, new_argc, plus_argv, 0, 0, &plugin_list);
506
348
    if (ret == ARGP_ERR_UNKNOWN){
507
349
      fprintf(stderr, "Unknown error while parsing arguments\n");
508
350
      exitstatus = EXIT_FAILURE;
509
 
      goto fallback;
 
351
      goto end;
510
352
    }
511
353
  }
512
354
  
517
359
      for(char **a = p->argv; *a != NULL; a++){
518
360
        fprintf(stderr, "\tArg: %s\n", *a);
519
361
      }
520
 
      fprintf(stderr, "...and %u environment variables\n", p->envc);
521
 
      for(char **a = p->environ; *a != NULL; a++){
522
 
        fprintf(stderr, "\t%s\n", *a);
523
 
      }
524
362
    }
525
363
  }
526
364
  
538
376
  if(dir == NULL){
539
377
    perror("Could not open plugin dir");
540
378
    exitstatus = EXIT_FAILURE;
541
 
    goto fallback;
 
379
    goto end;
542
380
  }
543
381
  
544
382
  /* Set the FD_CLOEXEC flag on the directory, if possible */
549
387
      if(ret < 0){
550
388
        perror("set_cloexec_flag");
551
389
        exitstatus = EXIT_FAILURE;
552
 
        goto fallback;
 
390
        goto end;
553
391
      }
554
392
    }
555
393
  }
564
402
      if (errno == EBADF){
565
403
        perror("readdir");
566
404
        exitstatus = EXIT_FAILURE;
567
 
        goto fallback;
 
405
        goto end;
568
406
      }
569
407
      break;
570
408
    }
615
453
        continue;
616
454
      }
617
455
    }
618
 
 
619
 
    char *filename;
620
 
    ret = asprintf(&filename, "%s/%s", plugindir, dirst->d_name);
621
 
    if(ret < 0){
622
 
      perror("asprintf");
 
456
    
 
457
    char *filename = malloc(d_name_len + strlen(plugindir) + 2);
 
458
    if (filename == NULL){
 
459
      perror("malloc");
623
460
      continue;
624
461
    }
 
462
    strcpy(filename, plugindir); /* Spurious warning */
 
463
    strcat(filename, "/");      /* Spurious warning */
 
464
    strcat(filename, dirst->d_name); /* Spurious warning */
625
465
    
626
466
    ret = stat(filename, &st);
627
467
    if (ret == -1){
638
478
      free(filename);
639
479
      continue;
640
480
    }
641
 
    plugin *p = getplugin(dirst->d_name, &plugin_list);
642
 
    if(p == NULL){
643
 
      perror("getplugin");
644
 
      free(filename);
645
 
      continue;
646
 
    }
647
 
    if(p->disabled){
 
481
    if(getplugin(dirst->d_name, &plugin_list)->disabled){
648
482
      if(debug){
649
483
        fprintf(stderr, "Ignoring disabled plugin \"%s\"\n",
650
484
                dirst->d_name);
652
486
      free(filename);
653
487
      continue;
654
488
    }
 
489
    plugin *p = getplugin(dirst->d_name, &plugin_list);
655
490
    {
656
491
      /* Add global arguments to argument list for this plugin */
657
492
      plugin *g = getplugin(NULL, &plugin_list);
658
 
      if(g != NULL){
659
 
        for(char **a = g->argv + 1; *a != NULL; a++){
660
 
          if(not add_argument(p, *a)){
661
 
            perror("add_argument");
662
 
          }
663
 
        }
664
 
        /* Add global environment variables */
665
 
        for(char **e = g->environ; *e != NULL; e++){
666
 
          if(not add_environment(p, *e)){
667
 
            perror("add_environment");
668
 
          }
669
 
        }
670
 
      }
671
 
    }
672
 
    /* If this plugin has any environment variables, we will call
673
 
       using execve and need to duplicate the environment from this
674
 
       process, too. */
675
 
    if(p->environ[0] != NULL){
676
 
      for(char **e = environ; *e != NULL; e++){
677
 
        char *copy = strdup(*e);
678
 
        if(copy == NULL){
679
 
          perror("strdup");
680
 
          continue;
681
 
        }
682
 
        if(not add_environment(p, copy)){
683
 
          perror("add_environment");
684
 
        }
685
 
      }
686
 
    }
687
 
    
 
493
      for(char **a = g->argv + 1; *a != NULL; a++){
 
494
        addargument(p, *a);
 
495
      }
 
496
    }
688
497
    int pipefd[2]; 
689
498
    ret = pipe(pipefd);
690
499
    if (ret == -1){
691
500
      perror("pipe");
692
501
      exitstatus = EXIT_FAILURE;
693
 
      goto fallback;
 
502
      goto end;
694
503
    }
695
504
    ret = set_cloexec_flag(pipefd[0]);
696
505
    if(ret < 0){
697
506
      perror("set_cloexec_flag");
698
507
      exitstatus = EXIT_FAILURE;
699
 
      goto fallback;
 
508
      goto end;
700
509
    }
701
510
    ret = set_cloexec_flag(pipefd[1]);
702
511
    if(ret < 0){
703
512
      perror("set_cloexec_flag");
704
513
      exitstatus = EXIT_FAILURE;
705
 
      goto fallback;
 
514
      goto end;
706
515
    }
707
516
    /* Block SIGCHLD until process is safely in process list */
708
517
    ret = sigprocmask (SIG_BLOCK, &sigchld_action.sa_mask, NULL);
709
518
    if(ret < 0){
710
519
      perror("sigprocmask");
711
520
      exitstatus = EXIT_FAILURE;
712
 
      goto fallback;
 
521
      goto end;
713
522
    }
714
523
    // Starting a new process to be watched
715
524
    pid_t pid = fork();
716
525
    if(pid == -1){
717
526
      perror("fork");
718
527
      exitstatus = EXIT_FAILURE;
719
 
      goto fallback;
 
528
      goto end;
720
529
    }
721
530
    if(pid == 0){
722
531
      /* this is the child process */
742
551
           above and must now close it manually here. */
743
552
        closedir(dir);
744
553
      }
745
 
      if(p->environ[0] == NULL){
746
 
        if(execv(filename, p->argv) < 0){
747
 
          perror("execv");
748
 
          _exit(EXIT_FAILURE);
749
 
        }
750
 
      } else {
751
 
        if(execve(filename, p->argv, p->environ) < 0){
752
 
          perror("execve");
753
 
          _exit(EXIT_FAILURE);
754
 
        }
 
554
      if(execv(filename, p->argv) < 0){
 
555
        perror("execv");
 
556
        _exit(EXIT_FAILURE);
755
557
      }
756
558
      /* no return */
757
559
    }
766
568
        perror("sigprocmask");
767
569
      }
768
570
      exitstatus = EXIT_FAILURE;
769
 
      goto fallback;
 
571
      goto end;
770
572
    }
771
573
    
772
574
    *new_process = (struct process){ .pid = pid,
780
582
    if(ret < 0){
781
583
      perror("sigprocmask");
782
584
      exitstatus = EXIT_FAILURE;
783
 
      goto fallback;
 
585
      goto end;
784
586
    }
785
587
    
786
588
    FD_SET(new_process->fd, &rfds_all);
791
593
    
792
594
  }
793
595
  
794
 
  free_plugin_list(plugin_list);
 
596
  /* Free the plugin list */
 
597
  for(plugin *next; plugin_list != NULL; plugin_list = next){
 
598
    next = plugin_list->next;
 
599
    free(plugin_list->argv);
 
600
    free(plugin_list);
 
601
  }
795
602
  
796
603
  closedir(dir);
797
604
  dir = NULL;
807
614
    if (select_ret == -1){
808
615
      perror("select");
809
616
      exitstatus = EXIT_FAILURE;
810
 
      goto fallback;
 
617
      goto end;
811
618
    }
812
619
    /* OK, now either a process completed, or something can be read
813
620
       from one of them */
839
646
          if(ret < 0){
840
647
            perror("sigprocmask");
841
648
            exitstatus = EXIT_FAILURE;
842
 
            goto fallback;
 
649
            goto end;
843
650
          }
844
651
          /* Delete this process entry from the list */
845
652
          if(process_list == proc){
874
681
          perror("print_out_password");
875
682
          exitstatus = EXIT_FAILURE;
876
683
        }
877
 
        goto fallback;
 
684
        goto end;
878
685
      }
879
686
      /* This process has not completed.  Does it have any output? */
880
687
      if(proc->eof or not FD_ISSET(proc->fd, &rfds)){
888
695
        if (proc->buffer == NULL){
889
696
          perror("malloc");
890
697
          exitstatus = EXIT_FAILURE;
891
 
          goto fallback;
 
698
          goto end;
892
699
        }
893
700
        proc->buffer_size += BUFFER_SIZE;
894
701
      }
909
716
  }
910
717
 
911
718
 
912
 
 fallback:
 
719
 end:
913
720
  
914
721
  if(process_list == NULL or exitstatus != EXIT_SUCCESS){
915
722
    /* Fallback if all plugins failed, none are found or an error occured */
920
727
    if(not bret){
921
728
      perror("print_out_password");
922
729
      exitstatus = EXIT_FAILURE;
 
730
      goto end;
923
731
    }
924
732
  }
925
733
  
926
734
  /* Restore old signal handler */
927
 
  ret = sigaction(SIGCHLD, &old_sigchld_action, NULL);
928
 
  if(ret == -1){
929
 
    perror("sigaction");
930
 
    exitstatus = EXIT_FAILURE;
931
 
  }
932
 
 
933
 
  if(custom_argv != NULL){
934
 
    for(char **arg = custom_argv; *arg != NULL; arg++){
935
 
      free(*arg);
936
 
    }
937
 
    free(custom_argv);
938
 
  }
939
 
  free_plugin_list(plugin_list);
 
735
  sigaction(SIGCHLD, &old_sigchld_action, NULL);
 
736
  
 
737
  free(plus_argv);
 
738
  
 
739
  /* Free the plugin list */
 
740
  for(plugin *next; plugin_list != NULL; plugin_list = next){
 
741
    next = plugin_list->next;
 
742
    free(plugin_list->argv);
 
743
    free(plugin_list);
 
744
  }
940
745
  
941
746
  if(dir != NULL){
942
747
    closedir(dir);