/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/password-prompt.c

  • Committer: Teddy Hogeborn
  • Date: 2017-06-24 10:08:21 UTC
  • Revision ID: teddy@recompile.se-20170624100821-3ojepr4r6sifqzj5
Fix memory leak in password-prompt

* plugins.d/password-prompt.c (conflict_detection): Free any
  individual direntries, not just the direntry list.

Show diffs side-by-side

added added

removed removed

Lines of Context:
plugins.d/password-prompt.c
2
2
/*
3
3
 * Password-prompt - Read a password from the terminal and print it
4
4
 * 
5
 
 * Copyright © 2008-2010 Teddy Hogeborn
6
 
 * Copyright © 2008-2010 Björn Påhlsson
 
5
 * Copyright © 2008-2017 Teddy Hogeborn
 
6
 * Copyright © 2008-2017 Björn Påhlsson
7
7
 * 
8
8
 * This program is free software: you can redistribute it and/or
9
9
 * modify it under the terms of the GNU General Public License as
19
19
 * along with this program.  If not, see
20
20
 * <http://www.gnu.org/licenses/>.
21
21
 * 
22
 
 * Contact the authors at <mandos@fukt.bsnet.se>.
 
22
 * Contact the authors at <mandos@recompile.se>.
23
23
 */
24
24
 
25
25
#define _GNU_SOURCE             /* getline(), asprintf() */
26
26
 
27
 
#include <termios.h>            /* struct termios, tcsetattr(),
 
27
#include <termios.h>            /* struct termios, tcsetattr(),
28
28
                                   TCSAFLUSH, tcgetattr(), ECHO */
29
29
#include <unistd.h>             /* struct termios, tcsetattr(),
30
30
                                   STDIN_FILENO, TCSAFLUSH,
41
41
                                   getenv(), free() */
42
42
#include <dirent.h>             /* scandir(), alphasort() */
43
43
#include <stdio.h>              /* fprintf(), stderr, getline(),
44
 
                                   stdin, feof(), fputc()
45
 
                                */
 
44
                                   stdin, feof(), fputc(), vfprintf(),
 
45
                                   vasprintf() */
46
46
#include <errno.h>              /* errno, EBADF, ENOTTY, EINVAL,
47
47
                                   EFAULT, EFBIG, EIO, ENOSPC, EINTR
48
48
                                */
50
50
#include <iso646.h>             /* or, not */
51
51
#include <stdbool.h>            /* bool, false, true */
52
52
#include <inttypes.h>           /* strtoumax() */
53
 
#include <sys/stat.h>           /* struct stat, lstat(), open() */
54
 
#include <string.h>             /* strlen, rindex, memcmp */
 
53
#include <sys/stat.h>           /* struct stat, lstat(), open() */
 
54
#include <string.h>             /* strlen, rindex, memcmp, strerror()
 
55
                                 */
55
56
#include <argp.h>               /* struct argp_option, struct
56
57
                                   argp_state, struct argp,
57
58
                                   argp_parse(), error_t,
60
61
#include <sysexits.h>           /* EX_SOFTWARE, EX_OSERR,
61
62
                                   EX_UNAVAILABLE, EX_IOERR, EX_OK */
62
63
#include <fcntl.h>              /* open() */
 
64
#include <stdarg.h>             /* va_list, va_start(), ... */
63
65
 
64
66
volatile sig_atomic_t quit_now = 0;
65
67
int signal_received;
66
68
bool debug = false;
67
69
const char *argp_program_version = "password-prompt " VERSION;
68
 
const char *argp_program_bug_address = "<mandos@fukt.bsnet.se>";
 
70
const char *argp_program_bug_address = "<mandos@recompile.se>";
69
71
 
70
72
/* Needed for conflict resolution */
71
73
const char plymouth_name[] = "plymouthd";
72
 
const char plymouth_alt_name[] = "plymouthd";
73
74
 
 
75
/* Function to use when printing errors */
 
76
__attribute__((format (gnu_printf, 3, 4)))
 
77
void error_plus(int status, int errnum, const char *formatstring,
 
78
                ...){
 
79
  va_list ap;
 
80
  char *text;
 
81
  int ret;
 
82
  
 
83
  va_start(ap, formatstring);
 
84
  ret = vasprintf(&text, formatstring, ap);
 
85
  if(ret == -1){
 
86
    fprintf(stderr, "Mandos plugin %s: ",
 
87
            program_invocation_short_name);
 
88
    vfprintf(stderr, formatstring, ap);
 
89
    fprintf(stderr, ": %s\n", strerror(errnum));
 
90
    error(status, errno, "vasprintf while printing error");
 
91
    return;
 
92
  }
 
93
  fprintf(stderr, "Mandos plugin ");
 
94
  error(status, errnum, "%s", text);
 
95
  free(text);
 
96
}
74
97
 
75
98
static void termination_handler(int signum){
76
99
  if(quit_now){
86
109
     from the terminal.  Password-prompt will exit if it detects
87
110
     plymouth since plymouth performs the same functionality.
88
111
   */
 
112
  __attribute__((nonnull))
89
113
  int is_plymouth(const struct dirent *proc_entry){
90
114
    int ret;
91
115
    int cl_fd;
92
116
    {
93
 
      uintmax_t maxvalue;
 
117
      uintmax_t proc_id;
94
118
      char *tmp;
95
119
      errno = 0;
96
 
      maxvalue = strtoumax(proc_entry->d_name, &tmp, 10);
 
120
      proc_id = strtoumax(proc_entry->d_name, &tmp, 10);
97
121
      
98
122
      if(errno != 0 or *tmp != '\0'
99
 
         or maxvalue != (uintmax_t)((pid_t)maxvalue)){
 
123
         or proc_id != (uintmax_t)((pid_t)proc_id)){
100
124
        return 0;
101
125
      }
102
126
    }
103
127
    
104
128
    char *cmdline_filename;
105
 
    ret = asprintf(&cmdline_filename, "/proc/%s/cmdline", proc_entry->d_name);
 
129
    ret = asprintf(&cmdline_filename, "/proc/%s/cmdline",
 
130
                   proc_entry->d_name);
106
131
    if(ret == -1){
107
 
      error(0, errno, "asprintf");
 
132
      error_plus(0, errno, "asprintf");
108
133
      return 0;
109
134
    }
110
135
    
111
 
    /* Open /proc/<pid>/cmdline  */
 
136
    /* Open /proc/<pid>/cmdline */
112
137
    cl_fd = open(cmdline_filename, O_RDONLY);
113
138
    free(cmdline_filename);
114
139
    if(cl_fd == -1){
115
 
      error(0, errno, "open");
 
140
      if(errno != ENOENT){
 
141
        error_plus(0, errno, "open");
 
142
      }
116
143
      return 0;
117
144
    }
118
145
    
128
155
        if(cmdline_len + blocksize + 1 > cmdline_allocated){
129
156
          tmp = realloc(cmdline, cmdline_allocated + blocksize + 1);
130
157
          if(tmp == NULL){
131
 
            error(0, errno, "realloc");
 
158
            error_plus(0, errno, "realloc");
132
159
            free(cmdline);
133
160
            close(cl_fd);
134
161
            return 0;
141
168
        sret = read(cl_fd, cmdline + cmdline_len,
142
169
                    cmdline_allocated - cmdline_len);
143
170
        if(sret == -1){
144
 
          error(0, errno, "read");
 
171
          error_plus(0, errno, "read");
145
172
          free(cmdline);
146
173
          close(cl_fd);
147
174
          return 0;
150
177
      } while(sret != 0);
151
178
      ret = close(cl_fd);
152
179
      if(ret == -1){
153
 
        error(0, errno, "close");
 
180
        error_plus(0, errno, "close");
154
181
        free(cmdline);
155
182
        return 0;
156
183
      }
166
193
      cmdline_base = cmdline;
167
194
    }
168
195
    
169
 
    if((strcmp(cmdline_base, plymouth_name) != 0)
170
 
       and (strcmp(cmdline_base, plymouth_alt_name) != 0)){
 
196
    if(strcmp(cmdline_base, plymouth_name) != 0){
 
197
      if(debug){
 
198
        fprintf(stderr, "\"%s\" is not \"%s\"\n", cmdline_base,
 
199
                plymouth_name);
 
200
      }
171
201
      free(cmdline);
172
202
      return 0;
173
203
    }
 
204
    if(debug){
 
205
      fprintf(stderr, "\"%s\" equals \"%s\"\n", cmdline_base,
 
206
              plymouth_name);
 
207
    }
174
208
    free(cmdline);
175
209
    return 1;
176
210
  }
177
 
 
178
 
  struct dirent **direntries;
 
211
  
 
212
  struct dirent **direntries = NULL;
179
213
  int ret;
180
214
  ret = scandir("/proc", &direntries, is_plymouth, alphasort);
181
 
  if (ret == -1){
182
 
    error(1, errno, "scandir");
183
 
  }
 
215
  if(ret == -1){
 
216
    error_plus(1, errno, "scandir");
 
217
  }
 
218
  {
 
219
    int i = ret;
 
220
    while(i--){
 
221
      free(direntries[i]);
 
222
    }
 
223
  }
 
224
  free(direntries);
184
225
  return ret > 0;
185
226
}
186
227
 
215
256
      { .name = NULL }
216
257
    };
217
258
    
 
259
    __attribute__((nonnull(3)))
218
260
    error_t parse_opt (int key, char *arg, struct argp_state *state){
219
261
      errno = 0;
220
262
      switch (key){
256
298
    case ENOMEM:
257
299
    default:
258
300
      errno = ret;
259
 
      error(0, errno, "argp_parse");
 
301
      error_plus(0, errno, "argp_parse");
260
302
      return EX_OSERR;
261
303
    case EINVAL:
262
304
      return EX_USAGE;
267
309
    fprintf(stderr, "Starting %s\n", argv[0]);
268
310
  }
269
311
 
270
 
  if (conflict_detection()){
 
312
  if(conflict_detection()){
271
313
    if(debug){
272
 
      fprintf(stderr, "Stopping %s because of conflict", argv[0]);
 
314
      fprintf(stderr, "Stopping %s because of conflict\n", argv[0]);
273
315
    }
274
316
    return EXIT_FAILURE;
275
317
  }
280
322
  
281
323
  if(tcgetattr(STDIN_FILENO, &t_old) != 0){
282
324
    int e = errno;
283
 
    error(0, errno, "tcgetattr");
 
325
    error_plus(0, errno, "tcgetattr");
284
326
    switch(e){
285
327
    case EBADF:
286
328
    case ENOTTY:
293
335
  sigemptyset(&new_action.sa_mask);
294
336
  ret = sigaddset(&new_action.sa_mask, SIGINT);
295
337
  if(ret == -1){
296
 
    error(0, errno, "sigaddset");
 
338
    error_plus(0, errno, "sigaddset");
297
339
    return EX_OSERR;
298
340
  }
299
341
  ret = sigaddset(&new_action.sa_mask, SIGHUP);
300
342
  if(ret == -1){
301
 
    error(0, errno, "sigaddset");
 
343
    error_plus(0, errno, "sigaddset");
302
344
    return EX_OSERR;
303
345
  }
304
346
  ret = sigaddset(&new_action.sa_mask, SIGTERM);
305
347
  if(ret == -1){
306
 
    error(0, errno, "sigaddset");
 
348
    error_plus(0, errno, "sigaddset");
307
349
    return EX_OSERR;
308
350
  }
309
351
  /* Need to check if the handler is SIG_IGN before handling:
312
354
  */
313
355
  ret = sigaction(SIGINT, NULL, &old_action);
314
356
  if(ret == -1){
315
 
    error(0, errno, "sigaction");
 
357
    error_plus(0, errno, "sigaction");
316
358
    return EX_OSERR;
317
359
  }
318
360
  if(old_action.sa_handler != SIG_IGN){
319
361
    ret = sigaction(SIGINT, &new_action, NULL);
320
362
    if(ret == -1){
321
 
      error(0, errno, "sigaction");
 
363
      error_plus(0, errno, "sigaction");
322
364
      return EX_OSERR;
323
365
    }
324
366
  }
325
367
  ret = sigaction(SIGHUP, NULL, &old_action);
326
368
  if(ret == -1){
327
 
    error(0, errno, "sigaction");
 
369
    error_plus(0, errno, "sigaction");
328
370
    return EX_OSERR;
329
371
  }
330
372
  if(old_action.sa_handler != SIG_IGN){
331
373
    ret = sigaction(SIGHUP, &new_action, NULL);
332
374
    if(ret == -1){
333
 
      error(0, errno, "sigaction");
 
375
      error_plus(0, errno, "sigaction");
334
376
      return EX_OSERR;
335
377
    }
336
378
  }
337
379
  ret = sigaction(SIGTERM, NULL, &old_action);
338
380
  if(ret == -1){
339
 
    error(0, errno, "sigaction");
 
381
    error_plus(0, errno, "sigaction");
340
382
    return EX_OSERR;
341
383
  }
342
384
  if(old_action.sa_handler != SIG_IGN){
343
385
    ret = sigaction(SIGTERM, &new_action, NULL);
344
386
    if(ret == -1){
345
 
      error(0, errno, "sigaction");
 
387
      error_plus(0, errno, "sigaction");
346
388
      return EX_OSERR;
347
389
    }
348
390
  }
356
398
  t_new.c_lflag &= ~(tcflag_t)ECHO;
357
399
  if(tcsetattr(STDIN_FILENO, TCSAFLUSH, &t_new) != 0){
358
400
    int e = errno;
359
 
    error(0, errno, "tcsetattr-echo");
 
401
    error_plus(0, errno, "tcsetattr-echo");
360
402
    switch(e){
361
403
    case EBADF:
362
404
    case ENOTTY:
426
468
        sret = write(STDOUT_FILENO, buffer + written, n - written);
427
469
        if(sret < 0){
428
470
          int e = errno;
429
 
          error(0, errno, "write");
 
471
          error_plus(0, errno, "write");
430
472
          switch(e){
431
473
          case EBADF:
432
474
          case EFAULT:
448
490
      sret = close(STDOUT_FILENO);
449
491
      if(sret == -1){
450
492
        int e = errno;
451
 
        error(0, errno, "close");
 
493
        error_plus(0, errno, "close");
452
494
        switch(e){
453
495
        case EBADF:
454
496
          status = EX_OSFILE;
464
506
    if(sret < 0){
465
507
      int e = errno;
466
508
      if(errno != EINTR and not feof(stdin)){
467
 
        error(0, errno, "getline");
 
509
        error_plus(0, errno, "getline");
468
510
        switch(e){
469
511
        case EBADF:
470
512
          status = EX_UNAVAILABLE;
 
513
          break;
471
514
        case EIO:
472
515
        case EINVAL:
473
516
        default:
477
520
        break;
478
521
      }
479
522
    }
480
 
    /* if(sret == 0), then the only sensible thing to do is to retry to
481
 
       read from stdin */
 
523
    /* if(sret == 0), then the only sensible thing to do is to retry
 
524
       to read from stdin */
482
525
    fputc('\n', stderr);
483
526
    if(debug and not quit_now){
484
527
      /* If quit_now is nonzero, we were interrupted by a signal, and
493
536
    fprintf(stderr, "Restoring terminal attributes\n");
494
537
  }
495
538
  if(tcsetattr(STDIN_FILENO, TCSAFLUSH, &t_old) != 0){
496
 
    error(0, errno, "tcsetattr+echo");
 
539
    error_plus(0, errno, "tcsetattr+echo");
497
540
  }
498
541
  
499
542
  if(quit_now){
501
544
    old_action.sa_handler = SIG_DFL;
502
545
    ret = sigaction(signal_received, &old_action, NULL);
503
546
    if(ret == -1){
504
 
      error(0, errno, "sigaction");
 
547
      error_plus(0, errno, "sigaction");
505
548
    }
506
549
    raise(signal_received);
507
550
  }