/mandos/trunk

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

« back to all changes in this revision

Viewing changes to plugin-runner.c

  • Committer: Teddy Hogeborn
  • Date: 2009-01-08 03:54:06 UTC
  • Revision ID: teddy@fukt.bsnet.se-20090108035406-01uqkv9xqgxlnw7c
Change the default value of the "checker" option command to make the
checker command be run-time dependent on the "host" client attribute
instead of reading it only once on startup:

* clients.conf (checker): Changed default commented-out value to
                          "fping -q -- %%(host)s".
* mandos (main): Changed default value for the "checker" config option
                 to "fping -q -- %%(host)s".
* mandos-clients.conf.xml (OPTIONS): Document changed default value.
  (EXAMPLE): Changed value of "checker".

* plugins.d/password-prompt.c: Whitespace changes only.

Show diffs side-by-side

added added

removed removed

Lines of Context:
plugin-runner.c
2
2
/*
3
3
 * Mandos plugin runner - Run Mandos plugins
4
4
 *
5
 
 * Copyright © 2007-2008 Teddy Hogeborn & Björn Påhlsson
 
5
 * Copyright © 2008,2009 Teddy Hogeborn
 
6
 * Copyright © 2008,2009 Björn Påhlsson
6
7
 * 
7
8
 * This program is free software: you can redistribute it and/or
8
9
 * modify it under the terms of the GNU General Public License as
27
28
#include <stdlib.h>             /* malloc(), exit(), EXIT_FAILURE,
28
29
                                   EXIT_SUCCESS, realloc() */
29
30
#include <stdbool.h>            /* bool, true, false */
30
 
#include <stdio.h>              /* perror, popen(), fileno(),
31
 
                                   fprintf(), stderr, STDOUT_FILENO */
 
31
#include <stdio.h>              /* perror, fileno(), fprintf(),
 
32
                                   stderr, STDOUT_FILENO */
32
33
#include <sys/types.h>          /* DIR, opendir(), stat(), struct
33
34
                                   stat, waitpid(), WIFEXITED(),
34
35
                                   WEXITSTATUS(), wait(), pid_t,
46
47
                                   fcntl(), setuid(), setgid(),
47
48
                                   F_GETFD, F_SETFD, FD_CLOEXEC,
48
49
                                   access(), pipe(), fork(), close()
49
 
                                   dup2, STDOUT_FILENO, _exit(),
 
50
                                   dup2(), STDOUT_FILENO, _exit(),
50
51
                                   execv(), write(), read(),
51
52
                                   close() */
52
53
#include <fcntl.h>              /* fcntl(), F_GETFD, F_SETFD,
69
70
#define PDIR "/lib/mandos/plugins.d"
70
71
#define AFILE "/conf/conf.d/mandos/plugin-runner.conf"
71
72
 
72
 
const char *argp_program_version = "plugin-runner 1.0";
 
73
const char *argp_program_version = "plugin-runner " VERSION;
73
74
const char *argp_program_bug_address = "<mandos@fukt.bsnet.se>";
74
75
 
75
 
struct plugin;
76
 
 
77
76
typedef struct plugin{
78
77
  char *name;                   /* can be NULL or any plugin name */
79
78
  char **argv;
96
95
 
97
96
static plugin *plugin_list = NULL;
98
97
 
99
 
/* Gets a existing plugin based on name,
 
98
/* Gets an existing plugin based on name,
100
99
   or if none is found, creates a new one */
101
100
static plugin *getplugin(char *name){
102
101
  /* Check for exiting plugin with that name */
118
117
      return NULL;
119
118
    }
120
119
  }
121
 
 
 
120
  
122
121
  *new_plugin = (plugin) { .name = copy_name,
123
122
                           .argc = 1,
124
123
                           .disabled = false,
132
131
  }
133
132
  new_plugin->argv[0] = copy_name;
134
133
  new_plugin->argv[1] = NULL;
135
 
 
 
134
  
136
135
  new_plugin->environ = malloc(sizeof(char *));
137
136
  if(new_plugin->environ == NULL){
138
137
    free(copy_name);
141
140
    return NULL;
142
141
  }
143
142
  new_plugin->environ[0] = NULL;
144
 
 
 
143
  
145
144
  /* Append the new plugin to the list */
146
145
  plugin_list = new_plugin;
147
146
  return new_plugin;
179
178
}
180
179
 
181
180
/* Add to a plugin's environment */
182
 
static bool add_environment(plugin *p, const char *def){
 
181
static bool add_environment(plugin *p, const char *def, bool replace){
183
182
  if(p == NULL){
184
183
    return false;
185
184
  }
 
185
  /* namelen = length of name of environment variable */
 
186
  size_t namelen = (size_t)(strchrnul(def, '=') - def);
 
187
  /* Search for this environment variable */
 
188
  for(char **e = p->environ; *e != NULL; e++){
 
189
    if(strncmp(*e, def, namelen + 1) == 0){
 
190
      /* It already exists */
 
191
      if(replace){
 
192
        char *new = realloc(*e, strlen(def) + 1);
 
193
        if(new == NULL){
 
194
          return false;
 
195
        }
 
196
        *e = new;
 
197
        strcpy(*e, def);
 
198
      }
 
199
      return true;
 
200
    }
 
201
  }
186
202
  return add_to_char_array(def, &(p->environ), &(p->envc));
187
203
}
188
204
 
191
207
 * Descriptor Flags".
192
208
 * *Note File Descriptor Flags:(libc)Descriptor Flags.
193
209
 */
194
 
static int set_cloexec_flag(int fd)
195
 
{
 
210
static int set_cloexec_flag(int fd){
196
211
  int ret = fcntl(fd, F_GETFD, 0);
197
212
  /* If reading the flags failed, return error indication now. */
198
213
  if(ret < 0){
205
220
 
206
221
/* Mark processes as completed when they exit, and save their exit
207
222
   status. */
208
 
void handle_sigchld(__attribute__((unused)) int sig){
 
223
static void handle_sigchld(__attribute__((unused)) int sig){
209
224
  while(true){
210
225
    plugin *proc = plugin_list;
211
226
    int status;
221
236
      /* No child processes */
222
237
      break;
223
238
    }
224
 
 
 
239
    
225
240
    /* A child exited, find it in process_list */
226
241
    while(proc != NULL and proc->pid != pid){
227
242
      proc = proc->next;
236
251
}
237
252
 
238
253
/* Prints out a password to stdout */
239
 
bool print_out_password(const char *buffer, size_t length){
 
254
static bool print_out_password(const char *buffer, size_t length){
240
255
  ssize_t ret;
241
 
  if(length>0 and buffer[length-1] == '\n'){
242
 
    length--;
243
 
  }
244
256
  for(size_t written = 0; written < length; written += (size_t)ret){
245
257
    ret = TEMP_FAILURE_RETRY(write(STDOUT_FILENO, buffer + written,
246
258
                                   length - written));
297
309
  struct stat st;
298
310
  fd_set rfds_all;
299
311
  int ret, maxfd = 0;
 
312
  ssize_t sret;
300
313
  uid_t uid = 65534;
301
314
  gid_t gid = 65534;
302
315
  bool debug = false;
327
340
    { .name = "global-options", .key = 'g',
328
341
      .arg = "OPTION[,OPTION[,...]]",
329
342
      .doc = "Options passed to all plugins" },
330
 
    { .name = "global-envs", .key = 'e',
 
343
    { .name = "global-env", .key = 'G',
331
344
      .arg = "VAR=value",
332
345
      .doc = "Environment variable passed to all plugins" },
333
346
    { .name = "options-for", .key = 'o',
334
347
      .arg = "PLUGIN:OPTION[,OPTION[,...]]",
335
348
      .doc = "Options passed only to specified plugin" },
336
 
    { .name = "envs-for", .key = 'f',
 
349
    { .name = "env-for", .key = 'E',
337
350
      .arg = "PLUGIN:ENV=value",
338
351
      .doc = "Environment variable passed to specified plugin" },
339
352
    { .name = "disable", .key = 'd',
340
353
      .arg = "PLUGIN",
341
354
      .doc = "Disable a specific plugin", .group = 1 },
 
355
    { .name = "enable", .key = 'e',
 
356
      .arg = "PLUGIN",
 
357
      .doc = "Enable a specific plugin", .group = 1 },
342
358
    { .name = "plugin-dir", .key = 128,
343
359
      .arg = "DIRECTORY",
344
360
      .doc = "Specify a different plugin directory", .group = 2 },
356
372
    { .name = NULL }
357
373
  };
358
374
  
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. */
 
375
  error_t parse_opt (int key, char *arg, __attribute__((unused))
 
376
                     struct argp_state *state) {
362
377
    switch (key) {
363
378
    case 'g':                   /* --global-options */
364
379
      if (arg != NULL){
374
389
        }
375
390
      }
376
391
      break;
377
 
    case 'e':                   /* --global-envs */
 
392
    case 'G':                   /* --global-env */
378
393
      if(arg == NULL){
379
394
        break;
380
395
      }
381
 
      {
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
 
        }
 
396
      if(not add_environment(getplugin(NULL), arg, true)){
 
397
        perror("add_environment");
389
398
      }
390
399
      break;
391
400
    case 'o':                   /* --options-for */
392
401
      if (arg != NULL){
393
402
        char *p_name = strsep(&arg, ":");
394
 
        if(p_name[0] == '\0'){
 
403
        if(p_name[0] == '\0' or arg == NULL){
395
404
          break;
396
405
        }
397
406
        char *opt = strsep(&arg, ":");
398
 
        if(opt[0] == '\0'){
 
407
        if(opt[0] == '\0' or opt == NULL){
399
408
          break;
400
409
        }
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
 
            }
 
410
        char *p;
 
411
        while((p = strsep(&opt, ",")) != NULL){
 
412
          if(p[0] == '\0'){
 
413
            continue;
 
414
          }
 
415
          if(not add_argument(getplugin(p_name), p)){
 
416
            perror("add_argument");
 
417
            return ARGP_ERR_UNKNOWN;
411
418
          }
412
419
        }
413
420
      }
414
421
      break;
415
 
    case 'f':                   /* --envs-for */
 
422
    case 'E':                   /* --env-for */
416
423
      if(arg == NULL){
417
424
        break;
418
425
      }
421
428
        if(envdef == NULL){
422
429
          break;
423
430
        }
424
 
        char *p_name = strndup(arg, (size_t) (envdef-arg));
425
 
        if(p_name == NULL){
426
 
          break;
427
 
        }
428
 
        envdef++;
429
 
        if(not add_environment(getplugin(p_name), envdef)){
 
431
        *envdef = '\0';
 
432
        if(not add_environment(getplugin(arg), envdef+1, true)){
430
433
          perror("add_environment");
431
434
        }
432
435
      }
440
443
        p->disabled = true;
441
444
      }
442
445
      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;
443
455
    case 128:                   /* --plugin-dir */
 
456
      free(plugindir);
444
457
      plugindir = strdup(arg);
445
458
      if(plugindir == NULL){
446
459
        perror("strdup");
447
460
      }      
448
461
      break;
449
462
    case 129:                   /* --config-file */
 
463
      /* This is already done by parse_opt_config_file() */
 
464
      break;
 
465
    case 130:                   /* --userid */
 
466
      uid = (uid_t)strtol(arg, NULL, 10);
 
467
      break;
 
468
    case 131:                   /* --groupid */
 
469
      gid = (gid_t)strtol(arg, NULL, 10);
 
470
      break;
 
471
    case 132:                   /* --debug */
 
472
      debug = true;
 
473
      break;
 
474
    case ARGP_KEY_ARG:
 
475
      /* Cryptsetup always passes an argument, which is an empty
 
476
         string if "none" was specified in /etc/crypttab.  So if
 
477
         argument was empty, we ignore it silently. */
 
478
      if(arg[0] != '\0'){
 
479
        fprintf(stderr, "Ignoring unknown argument \"%s\"\n", arg);
 
480
      }
 
481
      break;
 
482
    case ARGP_KEY_END:
 
483
      break;
 
484
    default:
 
485
      return ARGP_ERR_UNKNOWN;
 
486
    }
 
487
    return 0;
 
488
  }
 
489
  
 
490
  /* This option parser is the same as parse_opt() above, except it
 
491
     ignores everything but the --config-file option. */
 
492
  error_t parse_opt_config_file (int key, char *arg,
 
493
                                 __attribute__((unused))
 
494
                                 struct argp_state *state) {
 
495
    switch (key) {
 
496
    case 'g':                   /* --global-options */
 
497
    case 'G':                   /* --global-env */
 
498
    case 'o':                   /* --options-for */
 
499
    case 'E':                   /* --env-for */
 
500
    case 'd':                   /* --disable */
 
501
    case 'e':                   /* --enable */
 
502
    case 128:                   /* --plugin-dir */
 
503
      break;
 
504
    case 129:                   /* --config-file */
 
505
      free(argfile);
450
506
      argfile = strdup(arg);
451
507
      if(argfile == NULL){
452
508
        perror("strdup");
453
509
      }
454
510
      break;      
455
511
    case 130:                   /* --userid */
456
 
      uid = (uid_t)strtol(arg, NULL, 10);
457
 
      break;
458
512
    case 131:                   /* --groupid */
459
 
      gid = (gid_t)strtol(arg, NULL, 10);
460
 
      break;
461
513
    case 132:                   /* --debug */
462
 
      debug = true;
463
 
      break;
464
514
    case ARGP_KEY_ARG:
465
 
      fprintf(stderr, "Ignoring unknown argument \"%s\"\n", arg);
466
 
      break;
467
515
    case ARGP_KEY_END:
468
516
      break;
469
517
    default:
472
520
    return 0;
473
521
  }
474
522
  
475
 
  struct argp argp = { .options = options, .parser = parse_opt,
476
 
                       .args_doc = "[+PLUS_SEPARATED_OPTIONS]",
 
523
  struct argp argp = { .options = options,
 
524
                       .parser = parse_opt_config_file,
 
525
                       .args_doc = "",
477
526
                       .doc = "Mandos plugin runner -- Run plugins" };
478
527
  
479
 
  ret = argp_parse (&argp, argc, argv, 0, 0, NULL);
 
528
  /* Parse using the parse_opt_config_file in order to get the custom
 
529
     config file location, if any. */
 
530
  ret = argp_parse (&argp, argc, argv, ARGP_IN_ORDER, 0, NULL);
480
531
  if (ret == ARGP_ERR_UNKNOWN){
481
532
    fprintf(stderr, "Unknown error while parsing arguments\n");
482
533
    exitstatus = EXIT_FAILURE;
483
534
    goto fallback;
484
535
  }
485
 
 
486
 
  /* Opens the configfile if aviable */
 
536
  
 
537
  /* Reset to the normal argument parser */
 
538
  argp.parser = parse_opt;
 
539
  
 
540
  /* Open the configfile if available */
487
541
  if (argfile == NULL){
488
542
    conffp = fopen(AFILE, "r");
489
543
  } else {
493
547
    char *org_line = NULL;
494
548
    char *p, *arg, *new_arg, *line;
495
549
    size_t size = 0;
496
 
    ssize_t sret;
497
550
    const char whitespace_delims[] = " \r\t\f\v\n";
498
551
    const char comment_delim[] = "#";
499
552
 
507
560
    custom_argv[0] = argv[0];
508
561
    custom_argv[1] = NULL;
509
562
 
510
 
    /* for each line in the config file, strip whitespace and ignore commented text */
 
563
    /* for each line in the config file, strip whitespace and ignore
 
564
       commented text */
511
565
    while(true){
512
566
      sret = getline(&org_line, &size, conffp);
513
567
      if(sret == -1){
542
596
      }
543
597
    }
544
598
    free(org_line);
545
 
  } else{
 
599
  } else {
546
600
    /* Check for harmful errors and go to fallback. Other errors might
547
601
       not affect opening plugins */
548
602
    if (errno == EMFILE or errno == ENFILE or errno == ENOMEM){
554
608
  /* If there was any arguments from configuration file,
555
609
     pass them to parser as command arguments */
556
610
  if(custom_argv != NULL){
557
 
    ret = argp_parse (&argp, custom_argc, custom_argv, 0, 0, NULL);
 
611
    ret = argp_parse (&argp, custom_argc, custom_argv, ARGP_IN_ORDER,
 
612
                      0, NULL);
558
613
    if (ret == ARGP_ERR_UNKNOWN){
559
614
      fprintf(stderr, "Unknown error while parsing arguments\n");
560
615
      exitstatus = EXIT_FAILURE;
562
617
    }
563
618
  }
564
619
  
 
620
  /* Parse actual command line arguments, to let them override the
 
621
     config file */
 
622
  ret = argp_parse (&argp, argc, argv, ARGP_IN_ORDER, 0, NULL);
 
623
  if (ret == ARGP_ERR_UNKNOWN){
 
624
    fprintf(stderr, "Unknown error while parsing arguments\n");
 
625
    exitstatus = EXIT_FAILURE;
 
626
    goto fallback;
 
627
  }
 
628
  
565
629
  if(debug){
566
630
    for(plugin *p = plugin_list; p != NULL; p=p->next){
567
631
      fprintf(stderr, "Plugin: %s has %d arguments\n",
575
639
      }
576
640
    }
577
641
  }
578
 
 
 
642
  
579
643
  /* Strip permissions down to nobody */
580
644
  ret = setuid(uid);
581
645
  if (ret == -1){
585
649
  if (ret == -1){
586
650
    perror("setgid");
587
651
  }
588
 
 
 
652
  
589
653
  if (plugindir == NULL){
590
654
    dir = opendir(PDIR);
591
655
  } else {
612
676
  }
613
677
  
614
678
  FD_ZERO(&rfds_all);
615
 
 
 
679
  
616
680
  /* Read and execute any executable in the plugin directory*/
617
681
  while(true){
618
682
    dirst = readdir(dir);
619
683
    
620
 
    // All directory entries have been processed
 
684
    /* All directory entries have been processed */
621
685
    if(dirst == NULL){
622
686
      if (errno == EBADF){
623
687
        perror("readdir");
629
693
    
630
694
    d_name_len = strlen(dirst->d_name);
631
695
    
632
 
    // Ignore dotfiles, backup files and other junk
 
696
    /* Ignore dotfiles, backup files and other junk */
633
697
    {
634
698
      bool bad_name = false;
635
699
      
637
701
      
638
702
      const char const *bad_suffixes[] = { "~", "#", ".dpkg-new",
639
703
                                           ".dpkg-old",
 
704
                                           ".dpkg-bak",
640
705
                                           ".dpkg-divert", NULL };
641
706
      for(const char **pre = bad_prefixes; *pre != NULL; pre++){
642
707
        size_t pre_len = strlen(*pre);
673
738
    }
674
739
 
675
740
    char *filename;
676
 
    ret = asprintf(&filename, "%s/%s", plugindir, dirst->d_name);
 
741
    if(plugindir == NULL){
 
742
      ret = asprintf(&filename, PDIR "/%s", dirst->d_name);
 
743
    } else {
 
744
      ret = asprintf(&filename, "%s/%s", plugindir, dirst->d_name);
 
745
    }
677
746
    if(ret < 0){
678
747
      perror("asprintf");
679
748
      continue;
721
790
        }
722
791
        /* Add global environment variables */
723
792
        for(char **e = g->environ; *e != NULL; e++){
724
 
          if(not add_environment(p, *e)){
 
793
          if(not add_environment(p, *e, false)){
725
794
            perror("add_environment");
726
795
          }
727
796
        }
732
801
       process, too. */
733
802
    if(p->environ[0] != NULL){
734
803
      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)){
 
804
        if(not add_environment(p, *e, false)){
741
805
          perror("add_environment");
742
806
        }
743
807
      }
770
834
      exitstatus = EXIT_FAILURE;
771
835
      goto fallback;
772
836
    }
773
 
    // Starting a new process to be watched
 
837
    /* Starting a new process to be watched */
774
838
    pid_t pid = fork();
775
839
    if(pid == -1){
776
840
      perror("fork");
784
848
        perror("sigaction");
785
849
        _exit(EXIT_FAILURE);
786
850
      }
787
 
      ret = sigprocmask (SIG_UNBLOCK, &sigchld_action.sa_mask, NULL);
 
851
      ret = sigprocmask(SIG_UNBLOCK, &sigchld_action.sa_mask, NULL);
788
852
      if(ret < 0){
789
853
        perror("sigprocmask");
790
854
        _exit(EXIT_FAILURE);
791
855
      }
792
 
 
 
856
      
793
857
      ret = dup2(pipefd[1], STDOUT_FILENO); /* replace our stdout */
794
858
      if(ret == -1){
795
859
        perror("dup2");
845
909
    if (maxfd < new_plugin->fd){
846
910
      maxfd = new_plugin->fd;
847
911
    }
848
 
    
849
912
  }
850
913
  
851
914
  closedir(dir);
852
915
  dir = NULL;
853
 
 
 
916
  
854
917
  for(plugin *p = plugin_list; p != NULL; p = p->next){
855
918
    if(p->pid != 0){
856
919
      break;
861
924
      free_plugin_list();
862
925
    }
863
926
  }
864
 
 
 
927
  
865
928
  /* Main loop while running plugins exist */
866
929
  while(plugin_list){
867
930
    fd_set rfds = rfds_all;
873
936
    }
874
937
    /* OK, now either a process completed, or something can be read
875
938
       from one of them */
876
 
    for(plugin *proc = plugin_list; proc != NULL; proc = proc->next){
 
939
    for(plugin *proc = plugin_list; proc != NULL;){
877
940
      /* Is this process completely done? */
878
941
      if(proc->eof and proc->completed){
879
942
        /* Only accept the plugin output if it exited cleanly */
906
969
            exitstatus = EXIT_FAILURE;
907
970
            goto fallback;
908
971
          }
 
972
          
 
973
          plugin *next_plugin = proc->next;
909
974
          free_plugin(proc);
 
975
          proc = next_plugin;
 
976
          
910
977
          /* We are done modifying process list, so unblock signal */
911
978
          ret = sigprocmask (SIG_UNBLOCK, &sigchld_action.sa_mask,
912
979
                             NULL);
919
986
          if(plugin_list == NULL){
920
987
            break;
921
988
          }
 
989
          
922
990
          continue;
923
991
        }
924
992
        
925
993
        /* This process exited nicely, so print its buffer */
926
 
 
 
994
        
927
995
        bool bret = print_out_password(proc->buffer,
928
996
                                       proc->buffer_length);
929
997
        if(not bret){
936
1004
      /* This process has not completed.  Does it have any output? */
937
1005
      if(proc->eof or not FD_ISSET(proc->fd, &rfds)){
938
1006
        /* This process had nothing to say at this time */
 
1007
        proc = proc->next;
939
1008
        continue;
940
1009
      }
941
1010
      /* Before reading, make the process' data buffer large enough */
950
1019
        proc->buffer_size += BUFFER_SIZE;
951
1020
      }
952
1021
      /* Read from the process */
953
 
      ret = read(proc->fd, proc->buffer + proc->buffer_length,
954
 
                 BUFFER_SIZE);
955
 
      if(ret < 0){
 
1022
      sret = read(proc->fd, proc->buffer + proc->buffer_length,
 
1023
                  BUFFER_SIZE);
 
1024
      if(sret < 0){
956
1025
        /* Read error from this process; ignore the error */
 
1026
        proc = proc->next;
957
1027
        continue;
958
1028
      }
959
 
      if(ret == 0){
 
1029
      if(sret == 0){
960
1030
        /* got EOF */
961
1031
        proc->eof = true;
962
1032
      } else {
963
 
        proc->buffer_length += (size_t) ret;
 
1033
        proc->buffer_length += (size_t) sret;
964
1034
      }
965
1035
    }
966
1036
  }
974
1044
    bool bret;
975
1045
    fprintf(stderr, "Going to fallback mode using getpass(3)\n");
976
1046
    char *passwordbuffer = getpass("Password: ");
977
 
    bret = print_out_password(passwordbuffer, strlen(passwordbuffer));
 
1047
    size_t len = strlen(passwordbuffer);
 
1048
    /* Strip trailing newline */
 
1049
    if(len > 0 and passwordbuffer[len-1] == '\n'){
 
1050
      passwordbuffer[len-1] = '\0'; /* not strictly necessary */
 
1051
      len--;
 
1052
    }
 
1053
    bret = print_out_password(passwordbuffer, len);
978
1054
    if(not bret){
979
1055
      perror("print_out_password");
980
1056
      exitstatus = EXIT_FAILURE;
987
1063
    perror("sigaction");
988
1064
    exitstatus = EXIT_FAILURE;
989
1065
  }
990
 
 
 
1066
  
991
1067
  if(custom_argv != NULL){
992
1068
    for(char **arg = custom_argv+1; *arg != NULL; arg++){
993
1069
      free(*arg);
999
1075
    closedir(dir);
1000
1076
  }
1001
1077
  
1002
 
  /* Free the process list and kill the processes */
 
1078
  /* Kill the processes */
1003
1079
  for(plugin *p = plugin_list; p != NULL; p = p->next){
1004
1080
    if(p->pid != 0){
1005
1081
      close(p->fd);
1018
1094
  if(errno != ECHILD){
1019
1095
    perror("wait");
1020
1096
  }
1021
 
 
 
1097
  
1022
1098
  free_plugin_list();
1023
1099
  
1024
1100
  free(plugindir);