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

  • Committer: Teddy Hogeborn
  • Date: 2017-09-03 08:26:36 UTC
  • 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-2019 Teddy Hogeborn
6
 
 * Copyright © 2010-2019 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
 * 
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");
228
207
          and ((not interrupted_by_signal)
229
208
               or (not interruptable)));
230
209
  if(interrupted_by_signal and interruptable){
231
 
    if(debug){
232
 
      fprintf_plus(stderr, "Interrupted by signal\n");
233
 
    }
234
210
    return false;
235
211
  }
236
212
  if(ret == -1){
237
213
    error_plus(0, errno, "waitpid");
238
214
    return false;
239
215
  }
240
 
  if(debug){
241
 
    if(WIFEXITED(status)){
242
 
      fprintf_plus(stderr, "exec_and_wait exited: %d\n",
243
 
                   WEXITSTATUS(status));
244
 
    } else if(WIFSIGNALED(status)) {
245
 
      fprintf_plus(stderr, "exec_and_wait signaled: %d\n",
246
 
                   WTERMSIG(status));
247
 
    }
248
 
  }
249
216
  if(WIFEXITED(status) and (WEXITSTATUS(status) == 0)){
250
217
    return true;
251
218
  }
315
282
  }
316
283
  /* Try the old pid file location */
317
284
  if(proc_id == 0){
318
 
    pidfile = fopen(plymouth_old_pid, "r");
319
 
    if(pidfile != NULL){
320
 
      ret = fscanf(pidfile, "%" SCNuMAX, &proc_id);
321
 
      if(ret != 1){
322
 
        proc_id = 0;
323
 
      }
324
 
      fclose(pidfile);
325
 
    }
326
 
  }
327
 
  /* Try the old old pid file location */
328
 
  if(proc_id == 0){
329
 
    pidfile = fopen(plymouth_old_old_pid, "r");
 
285
    pidfile = fopen(plymouth_pid, "r");
330
286
    if(pidfile != NULL){
331
287
      ret = fscanf(pidfile, "%" SCNuMAX, &proc_id);
332
288
      if(ret != 1){
343
299
      error_plus(0, errno, "scandir");
344
300
    }
345
301
    if(ret > 0){
346
 
      for(int i = ret-1; i >= 0; i--){
347
 
        if(proc_id == 0){
348
 
          ret = sscanf(direntries[i]->d_name, "%" SCNuMAX, &proc_id);
349
 
          if(ret < 0){
350
 
            error_plus(0, errno, "sscanf");
351
 
          }
352
 
        }
353
 
        free(direntries[i]);
 
302
      ret = sscanf(direntries[0]->d_name, "%" SCNuMAX, &proc_id);
 
303
      if(ret < 0){
 
304
        error_plus(0, errno, "sscanf");
354
305
      }
355
306
    }
356
307
    /* scandir might preallocate for this variable (man page unclear).
438
389
 
439
390
int main(__attribute__((unused))int argc,
440
391
         __attribute__((unused))char **argv){
441
 
  char *prompt = NULL;
 
392
  char *prompt;
442
393
  char *prompt_arg;
443
394
  pid_t plymouth_command_pid;
444
395
  int ret;
445
396
  bool bret;
446
397
 
447
 
  {
448
 
    struct argp_option options[] = {
449
 
      { .name = "prompt", .key = 128, .arg = "PROMPT",
450
 
        .doc = "The prompt to show" },
451
 
      { .name = "debug", .key = 129,
452
 
        .doc = "Debug mode" },
453
 
      { .name = NULL }
454
 
    };
455
 
    
456
 
    __attribute__((nonnull(3)))
457
 
    error_t parse_opt (int key, char *arg, __attribute__((unused))
458
 
                       struct argp_state *state){
459
 
      errno = 0;
460
 
      switch (key){
461
 
      case 128:                 /* --prompt */
462
 
        prompt = arg;
463
 
        if(debug){
464
 
          fprintf_plus(stderr, "Custom prompt \"%s\"\n", prompt);
465
 
        }
466
 
        break;
467
 
      case 129:                 /* --debug */
468
 
        debug = true;
469
 
        break;
470
 
      default:
471
 
        return ARGP_ERR_UNKNOWN;
472
 
      }
473
 
      return errno;
474
 
    }
475
 
    
476
 
    struct argp argp = { .options = options, .parser = parse_opt,
477
 
                         .args_doc = "",
478
 
                         .doc = "Mandos plymouth -- Read and"
479
 
                         " output a password" };
480
 
    ret = argp_parse(&argp, argc, argv, ARGP_IN_ORDER, NULL, NULL);
481
 
    switch(ret){
482
 
    case 0:
483
 
      break;
484
 
    case ENOMEM:
485
 
    default:
486
 
      errno = ret;
487
 
      error_plus(0, errno, "argp_parse");
488
 
      return EX_OSERR;
489
 
    case EINVAL:
490
 
      error_plus(0, errno, "argp_parse");
491
 
      return EX_USAGE;
492
 
    }
493
 
  }
494
 
  
495
398
  /* test -x /bin/plymouth */
496
399
  ret = access(plymouth_path, X_OK);
497
400
  if(ret == -1){
498
401
    /* Plymouth is probably not installed.  Don't print an error
499
402
       message, just exit. */
500
 
    if(debug){
501
 
      fprintf_plus(stderr, "Plymouth (%s) not found\n",
502
 
                   plymouth_path);
503
 
    }
504
403
    exit(EX_UNAVAILABLE);
505
404
  }
506
405
  
540
439
    }
541
440
    /* Plymouth is probably not running.  Don't print an error
542
441
       message, just exit. */
543
 
    if(debug){
544
 
      fprintf_plus(stderr, "Plymouth not running\n");
545
 
    }
546
442
    exit(EX_UNAVAILABLE);
547
443
  }
548
444
  
549
 
  if(prompt != NULL){
550
 
    ret = asprintf(&prompt_arg, "--prompt=%s", prompt);
551
 
  } else {
552
 
    char *made_prompt = makeprompt();
553
 
    ret = asprintf(&prompt_arg, "--prompt=%s", made_prompt);
554
 
    free(made_prompt);
555
 
  }
 
445
  prompt = makeprompt();
 
446
  ret = asprintf(&prompt_arg, "--prompt=%s", prompt);
 
447
  free(prompt);
556
448
  if(ret == -1){
557
449
    error_plus(EX_OSERR, errno, "asprintf");
558
450
  }
559
451
  
560
452
  /* plymouth ask-for-password --prompt="$prompt" */
561
 
  if(debug){
562
 
    fprintf_plus(stderr, "Prompting for password via Plymouth\n");
563
 
  }
564
453
  bret = exec_and_wait(&plymouth_command_pid,
565
454
                       plymouth_path, (const char *[])
566
455
                       { plymouth_path, "ask-for-password",