/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 plugins.d/plymouth.c

  • Committer: Teddy Hogeborn
  • Date: 2017-09-03 08:26:36 UTC
  • mto: (237.7.594 trunk)
  • mto: This revision was merged to the branch mainline in revision 362.
  • Revision ID: teddy@recompile.se-20170903082636-fo3qw1uoahjxw0by
Fix memory leak in plymouth plugin

* plugins.d/plymouth.c (getargv): Change return type to non-const.
  (main/plymouth_args): Change type to non-const and default to NULL.
  (main): Don't change plymouthd_argv, instead conditionally use it if
  it is allocated or use plymouthd_default_argv if not.
  (main): Free plymouthd_argv, if allocated.

Show diffs side-by-side

added added

removed removed

Lines of Context:
plugins.d/plymouth.c
2
2
/*
3
3
 * Plymouth - Read a password from Plymouth and output it
4
4
 * 
5
 
 * Copyright © 2010-2020 Teddy Hogeborn
6
 
 * Copyright © 2010-2020 Björn Påhlsson
 
5
 * Copyright © 2010-2017 Teddy Hogeborn
 
6
 * Copyright © 2010-2017 Björn Påhlsson
7
7
 * 
8
8
 * This file is part of Mandos.
9
9
 * 
44
44
                                   STDERR_FILENO, execv(), access() */
45
45
#include <stdlib.h>             /* free(), EXIT_FAILURE, realloc(),
46
46
                                   EXIT_SUCCESS, malloc(), _exit(),
47
 
                                   getenv(), reallocarray() */
 
47
                                   getenv() */
48
48
#include <dirent.h>             /* scandir(), alphasort() */
49
49
#include <inttypes.h>           /* intmax_t, strtoumax(), SCNuMAX */
50
50
#include <sys/stat.h>           /* struct stat, lstat() */
53
53
#include <errno.h>              /* TEMP_FAILURE_RETRY */
54
54
#include <argz.h>               /* argz_count(), argz_extract() */
55
55
#include <stdarg.h>             /* va_list, va_start(), ... */
56
 
#include <argp.h>
57
56
 
58
57
sig_atomic_t interrupted_by_signal = 0;
59
 
const char *argp_program_version = "plymouth " VERSION;
60
 
const char *argp_program_bug_address = "<mandos@recompile.se>";
61
58
 
62
59
/* Used by Ubuntu 11.04 (Natty Narwahl) */
63
 
const char plymouth_old_old_pid[] = "/dev/.initramfs/plymouth.pid";
 
60
const char plymouth_old_pid[] = "/dev/.initramfs/plymouth.pid";
64
61
/* Used by Ubuntu 11.10 (Oneiric Ocelot) */
65
 
const char plymouth_old_pid[] = "/run/initramfs/plymouth.pid";
66
 
/* Used by Debian 9 (stretch) */
67
 
const char plymouth_pid[] = "/run/plymouth/pid";
 
62
const char plymouth_pid[] = "/run/initramfs/plymouth.pid";
68
63
 
69
64
const char plymouth_path[] = "/bin/plymouth";
70
65
const char plymouthd_path[] = "/sbin/plymouthd";
72
67
                                        "--mode=boot",
73
68
                                        "--attach-to-session",
74
69
                                        NULL };
75
 
bool debug = false;
76
70
 
77
71
static void termination_handler(__attribute__((unused))int signum){
78
72
  if(interrupted_by_signal){
81
75
  interrupted_by_signal = 1;
82
76
}
83
77
 
84
 
__attribute__((format (gnu_printf, 2, 3), nonnull))
85
 
int fprintf_plus(FILE *stream, const char *format, ...){
86
 
  va_list ap;
87
 
  va_start (ap, format);
88
 
  fprintf(stream, "Mandos plugin %s: ", program_invocation_short_name);
89
 
  return vfprintf(stream, format, ap);
90
 
}
91
 
 
92
78
/* Function to use when printing errors */
93
79
__attribute__((format (gnu_printf, 3, 4)))
94
80
void error_plus(int status, int errnum, const char *formatstring,
171
157
 
172
158
__attribute__((nonnull (2, 3)))
173
159
bool exec_and_wait(pid_t *pid_return, const char *path,
174
 
                   const char * const * const argv, bool interruptable,
 
160
                   const char * const *argv, bool interruptable,
175
161
                   bool daemonize){
176
162
  int status;
177
163
  int ret;
178
164
  pid_t pid;
179
 
  if(debug){
180
 
    for(const char * const *arg = argv; *arg != NULL; arg++){
181
 
      fprintf_plus(stderr, "exec_and_wait arg: %s\n", *arg);
182
 
    }
183
 
    fprintf_plus(stderr, "exec_and_wait end of args\n");
184
 
  }
185
 
 
186
165
  pid = fork();
187
166
  if(pid == -1){
188
167
    error_plus(0, errno, "fork");
204
183
    char **tmp;
205
184
    int i = 0;
206
185
    for (; argv[i] != NULL; i++){
207
 
#if defined(__GLIBC_PREREQ) and __GLIBC_PREREQ(2, 26)
208
 
      tmp = reallocarray(new_argv, ((size_t)i + 2),
209
 
                         sizeof(const char *));
210
 
#else
211
 
      if(((size_t)i + 2) > (SIZE_MAX / sizeof(const char *))){
212
 
        /* overflow */
213
 
        tmp = NULL;
214
 
        errno = ENOMEM;
215
 
      } else {
216
 
        tmp = realloc(new_argv, ((size_t)i + 2) * sizeof(const char *));
217
 
      }
218
 
#endif
 
186
      tmp = realloc(new_argv, sizeof(const char *) * ((size_t)i + 2));
219
187
      if(tmp == NULL){
220
 
        error_plus(0, errno, "reallocarray");
 
188
        error_plus(0, errno, "realloc");
221
189
        free(new_argv);
222
190
        _exit(EX_OSERR);
223
191
      }
239
207
          and ((not interrupted_by_signal)
240
208
               or (not interruptable)));
241
209
  if(interrupted_by_signal and interruptable){
242
 
    if(debug){
243
 
      fprintf_plus(stderr, "Interrupted by signal\n");
244
 
    }
245
210
    return false;
246
211
  }
247
212
  if(ret == -1){
248
213
    error_plus(0, errno, "waitpid");
249
214
    return false;
250
215
  }
251
 
  if(debug){
252
 
    if(WIFEXITED(status)){
253
 
      fprintf_plus(stderr, "exec_and_wait exited: %d\n",
254
 
                   WEXITSTATUS(status));
255
 
    } else if(WIFSIGNALED(status)) {
256
 
      fprintf_plus(stderr, "exec_and_wait signaled: %d\n",
257
 
                   WTERMSIG(status));
258
 
    }
259
 
  }
260
216
  if(WIFEXITED(status) and (WEXITSTATUS(status) == 0)){
261
217
    return true;
262
218
  }
326
282
  }
327
283
  /* Try the old pid file location */
328
284
  if(proc_id == 0){
329
 
    pidfile = fopen(plymouth_old_pid, "r");
330
 
    if(pidfile != NULL){
331
 
      ret = fscanf(pidfile, "%" SCNuMAX, &proc_id);
332
 
      if(ret != 1){
333
 
        proc_id = 0;
334
 
      }
335
 
      fclose(pidfile);
336
 
    }
337
 
  }
338
 
  /* Try the old old pid file location */
339
 
  if(proc_id == 0){
340
 
    pidfile = fopen(plymouth_old_old_pid, "r");
 
285
    pidfile = fopen(plymouth_pid, "r");
341
286
    if(pidfile != NULL){
342
287
      ret = fscanf(pidfile, "%" SCNuMAX, &proc_id);
343
288
      if(ret != 1){
354
299
      error_plus(0, errno, "scandir");
355
300
    }
356
301
    if(ret > 0){
357
 
      for(int i = ret-1; i >= 0; i--){
358
 
        if(proc_id == 0){
359
 
          ret = sscanf(direntries[i]->d_name, "%" SCNuMAX, &proc_id);
360
 
          if(ret < 0){
361
 
            error_plus(0, errno, "sscanf");
362
 
          }
363
 
        }
364
 
        free(direntries[i]);
 
302
      ret = sscanf(direntries[0]->d_name, "%" SCNuMAX, &proc_id);
 
303
      if(ret < 0){
 
304
        error_plus(0, errno, "sscanf");
365
305
      }
366
306
    }
367
307
    /* scandir might preallocate for this variable (man page unclear).
449
389
 
450
390
int main(__attribute__((unused))int argc,
451
391
         __attribute__((unused))char **argv){
452
 
  char *prompt = NULL;
 
392
  char *prompt;
453
393
  char *prompt_arg;
454
394
  pid_t plymouth_command_pid;
455
395
  int ret;
456
396
  bool bret;
457
397
 
458
 
  {
459
 
    struct argp_option options[] = {
460
 
      { .name = "prompt", .key = 128, .arg = "PROMPT",
461
 
        .doc = "The prompt to show" },
462
 
      { .name = "debug", .key = 129,
463
 
        .doc = "Debug mode" },
464
 
      { .name = NULL }
465
 
    };
466
 
    
467
 
    __attribute__((nonnull(3)))
468
 
    error_t parse_opt (int key, char *arg, __attribute__((unused))
469
 
                       struct argp_state *state){
470
 
      errno = 0;
471
 
      switch (key){
472
 
      case 128:                 /* --prompt */
473
 
        prompt = arg;
474
 
        if(debug){
475
 
          fprintf_plus(stderr, "Custom prompt \"%s\"\n", prompt);
476
 
        }
477
 
        break;
478
 
      case 129:                 /* --debug */
479
 
        debug = true;
480
 
        break;
481
 
      default:
482
 
        return ARGP_ERR_UNKNOWN;
483
 
      }
484
 
      return errno;
485
 
    }
486
 
    
487
 
    struct argp argp = { .options = options, .parser = parse_opt,
488
 
                         .args_doc = "",
489
 
                         .doc = "Mandos plymouth -- Read and"
490
 
                         " output a password" };
491
 
    ret = argp_parse(&argp, argc, argv, ARGP_IN_ORDER, NULL, NULL);
492
 
    switch(ret){
493
 
    case 0:
494
 
      break;
495
 
    case ENOMEM:
496
 
    default:
497
 
      errno = ret;
498
 
      error_plus(0, errno, "argp_parse");
499
 
      return EX_OSERR;
500
 
    case EINVAL:
501
 
      error_plus(0, errno, "argp_parse");
502
 
      return EX_USAGE;
503
 
    }
504
 
  }
505
 
  
506
398
  /* test -x /bin/plymouth */
507
399
  ret = access(plymouth_path, X_OK);
508
400
  if(ret == -1){
509
401
    /* Plymouth is probably not installed.  Don't print an error
510
402
       message, just exit. */
511
 
    if(debug){
512
 
      fprintf_plus(stderr, "Plymouth (%s) not found\n",
513
 
                   plymouth_path);
514
 
    }
515
403
    exit(EX_UNAVAILABLE);
516
404
  }
517
405
  
551
439
    }
552
440
    /* Plymouth is probably not running.  Don't print an error
553
441
       message, just exit. */
554
 
    if(debug){
555
 
      fprintf_plus(stderr, "Plymouth not running\n");
556
 
    }
557
442
    exit(EX_UNAVAILABLE);
558
443
  }
559
444
  
560
 
  if(prompt != NULL){
561
 
    ret = asprintf(&prompt_arg, "--prompt=%s", prompt);
562
 
  } else {
563
 
    char *made_prompt = makeprompt();
564
 
    ret = asprintf(&prompt_arg, "--prompt=%s", made_prompt);
565
 
    free(made_prompt);
566
 
  }
 
445
  prompt = makeprompt();
 
446
  ret = asprintf(&prompt_arg, "--prompt=%s", prompt);
 
447
  free(prompt);
567
448
  if(ret == -1){
568
449
    error_plus(EX_OSERR, errno, "asprintf");
569
450
  }
570
451
  
571
452
  /* plymouth ask-for-password --prompt="$prompt" */
572
 
  if(debug){
573
 
    fprintf_plus(stderr, "Prompting for password via Plymouth\n");
574
 
  }
575
453
  bret = exec_and_wait(&plymouth_command_pid,
576
454
                       plymouth_path, (const char *[])
577
455
                       { plymouth_path, "ask-for-password",