/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: 2019-07-14 22:39:15 UTC
  • Revision ID: teddy@recompile.se-20190714223915-aqjkms3t3taa6tye
Only use sanitizing options when debugging

The C compiler's sanitizing options introduce code in the output
binary which is fragile and not very security conscious.  It has
become clear that sanitizing is only really meant for use while
debugging.

As a side effect, this makes compilation faster, as the Makefile, for
production builds, no longer runs the compiler repeatedly to find all
its currently supported sanitizing options.

* Makefile (DEBUG): Add "$(SANITIZE)".
  (SANITIZE): Comment out.
  (CFLAGS): Remove "$(SANITIZE)".
  (plugins.d/mandos-client): Revert back to use plain $(LINK.c), since
                             we no longer need to remove the leak
                             sanitizer by overriding CFLAGS.

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
7
 
 * 
8
 
 * This program is free software: you can redistribute it and/or
9
 
 * modify it under the terms of the GNU General Public License as
10
 
 * published by the Free Software Foundation, either version 3 of the
11
 
 * License, or (at your option) any later version.
12
 
 * 
13
 
 * This program is distributed in the hope that it will be useful, but
 
5
 * Copyright © 2008-2019 Teddy Hogeborn
 
6
 * Copyright © 2008-2019 Björn Påhlsson
 
7
 * 
 
8
 * This file is part of Mandos.
 
9
 * 
 
10
 * Mandos is free software: you can redistribute it and/or modify it
 
11
 * under the terms of the GNU General Public License as published by
 
12
 * the Free Software Foundation, either version 3 of the License, or
 
13
 * (at your option) any later version.
 
14
 * 
 
15
 * Mandos is distributed in the hope that it will be useful, but
14
16
 * WITHOUT ANY WARRANTY; without even the implied warranty of
15
17
 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
16
18
 * General Public License for more details.
17
19
 * 
18
20
 * You should have received a copy of the GNU General Public License
19
 
 * along with this program.  If not, see
20
 
 * <http://www.gnu.org/licenses/>.
 
21
 * along with Mandos.  If not, see <http://www.gnu.org/licenses/>.
21
22
 * 
22
 
 * Contact the authors at <mandos@fukt.bsnet.se>.
 
23
 * Contact the authors at <mandos@recompile.se>.
23
24
 */
24
25
 
25
26
#define _GNU_SOURCE             /* getline(), asprintf() */
26
27
 
27
 
#include <termios.h>            /* struct termios, tcsetattr(),
 
28
#include <termios.h>            /* struct termios, tcsetattr(),
28
29
                                   TCSAFLUSH, tcgetattr(), ECHO */
29
30
#include <unistd.h>             /* struct termios, tcsetattr(),
30
31
                                   STDIN_FILENO, TCSAFLUSH,
35
36
                                   SIGQUIT, SIGHUP, SIGTERM,
36
37
                                   raise() */
37
38
#include <stddef.h>             /* NULL, size_t, ssize_t */
38
 
#include <sys/types.h>          /* ssize_t, struct dirent, pid_t, ssize_t */
 
39
#include <sys/types.h>          /* ssize_t, struct dirent, pid_t,
 
40
                                   ssize_t, open() */
39
41
#include <stdlib.h>             /* EXIT_SUCCESS, EXIT_FAILURE,
40
42
                                   getenv(), free() */
41
43
#include <dirent.h>             /* scandir(), alphasort() */
42
44
#include <stdio.h>              /* fprintf(), stderr, getline(),
43
 
                                   stdin, feof(), fputc()
44
 
                                */
 
45
                                   stdin, feof(), fputc(), vfprintf(),
 
46
                                   vasprintf() */
45
47
#include <errno.h>              /* errno, EBADF, ENOTTY, EINVAL,
46
48
                                   EFAULT, EFBIG, EIO, ENOSPC, EINTR
47
49
                                */
49
51
#include <iso646.h>             /* or, not */
50
52
#include <stdbool.h>            /* bool, false, true */
51
53
#include <inttypes.h>           /* strtoumax() */
52
 
#include <sys/stat.h>           /* struct stat, lstat() */
53
 
#include <string.h>             /* strlen, rindex, memcmp */
 
54
#include <sys/stat.h>           /* struct stat, lstat(), open() */
 
55
#include <string.h>             /* strlen, rindex, memcmp, strerror()
 
56
                                 */
54
57
#include <argp.h>               /* struct argp_option, struct
55
58
                                   argp_state, struct argp,
56
59
                                   argp_parse(), error_t,
58
61
                                   ARGP_ERR_UNKNOWN */
59
62
#include <sysexits.h>           /* EX_SOFTWARE, EX_OSERR,
60
63
                                   EX_UNAVAILABLE, EX_IOERR, EX_OK */
 
64
#include <fcntl.h>              /* open() */
 
65
#include <stdarg.h>             /* va_list, va_start(), ... */
61
66
 
62
67
volatile sig_atomic_t quit_now = 0;
63
68
int signal_received;
64
69
bool debug = false;
65
70
const char *argp_program_version = "password-prompt " VERSION;
66
 
const char *argp_program_bug_address = "<mandos@fukt.bsnet.se>";
67
 
 
68
 
/* Needed for conflic resolution */
69
 
const char plymouthd_path[] = "/sbin/plymouth";
70
 
 
 
71
const char *argp_program_bug_address = "<mandos@recompile.se>";
 
72
 
 
73
/* Needed for conflict resolution */
 
74
const char plymouth_name[] = "plymouthd";
 
75
 
 
76
/* Function to use when printing errors */
 
77
__attribute__((format (gnu_printf, 3, 4)))
 
78
void error_plus(int status, int errnum, const char *formatstring,
 
79
                ...){
 
80
  va_list ap;
 
81
  char *text;
 
82
  int ret;
 
83
  
 
84
  va_start(ap, formatstring);
 
85
  ret = vasprintf(&text, formatstring, ap);
 
86
  if(ret == -1){
 
87
    fprintf(stderr, "Mandos plugin %s: ",
 
88
            program_invocation_short_name);
 
89
    vfprintf(stderr, formatstring, ap);
 
90
    fprintf(stderr, ": %s\n", strerror(errnum));
 
91
    error(status, errno, "vasprintf while printing error");
 
92
    return;
 
93
  }
 
94
  fprintf(stderr, "Mandos plugin ");
 
95
  error(status, errnum, "%s", text);
 
96
  free(text);
 
97
}
71
98
 
72
99
static void termination_handler(int signum){
73
100
  if(quit_now){
79
106
 
80
107
bool conflict_detection(void){
81
108
 
82
 
  /* plymouth conflicts with password-promt since both want to control the
83
 
     associated terminal. Password-prompt exit since plymouth perferms the same
84
 
     functionallity.
 
109
  /* plymouth conflicts with password-prompt since both want to read
 
110
     from the terminal.  Password-prompt will exit if it detects
 
111
     plymouth since plymouth performs the same functionality.
85
112
   */
 
113
  __attribute__((nonnull))
86
114
  int is_plymouth(const struct dirent *proc_entry){
87
115
    int ret;
 
116
    int cl_fd;
88
117
    {
89
 
      uintmax_t maxvalue;
 
118
      uintmax_t proc_id;
90
119
      char *tmp;
91
120
      errno = 0;
92
 
      maxvalue = strtoumax(proc_entry->d_name, &tmp, 10);
 
121
      proc_id = strtoumax(proc_entry->d_name, &tmp, 10);
93
122
      
94
123
      if(errno != 0 or *tmp != '\0'
95
 
         or maxvalue != (uintmax_t)((pid_t)maxvalue)){
 
124
         or proc_id != (uintmax_t)((pid_t)proc_id)){
96
125
        return 0;
97
126
      }
98
127
    }
99
 
    char exe_target[sizeof(plymouthd_path)];
100
 
    char *exe_link;
101
 
    ret = asprintf(&exe_link, "/proc/%s/exe", proc_entry->d_name);
 
128
    
 
129
    char *cmdline_filename;
 
130
    ret = asprintf(&cmdline_filename, "/proc/%s/cmdline",
 
131
                   proc_entry->d_name);
102
132
    if(ret == -1){
103
 
      error(0, errno, "asprintf");
 
133
      error_plus(0, errno, "asprintf");
104
134
      return 0;
105
135
    }
106
 
  
107
 
    struct stat exe_stat;
108
 
    ret = lstat(exe_link, &exe_stat);
109
 
    if(ret == -1){
110
 
      free(exe_link);
 
136
    
 
137
    /* Open /proc/<pid>/cmdline */
 
138
    cl_fd = open(cmdline_filename, O_RDONLY);
 
139
    free(cmdline_filename);
 
140
    if(cl_fd == -1){
111
141
      if(errno != ENOENT){
112
 
        error(0, errno, "lstat");
113
 
      }
114
 
      return 0;
115
 
    }
116
 
  
117
 
    if(not S_ISLNK(exe_stat.st_mode)
118
 
       or exe_stat.st_uid != 0
119
 
       or exe_stat.st_gid != 0){
120
 
      free(exe_link);
121
 
      return 0;
122
 
    }
123
 
  
124
 
    ssize_t sret = readlink(exe_link, exe_target, sizeof(exe_target));
125
 
    free(exe_link);
126
 
    if((sret != (ssize_t)sizeof(plymouthd_path)-1) or
127
 
       (memcmp(plymouthd_path, exe_target,
128
 
               sizeof(plymouthd_path)-1) != 0)){
129
 
      return 0;
130
 
    }
 
142
        error_plus(0, errno, "open");
 
143
      }
 
144
      return 0;
 
145
    }
 
146
    
 
147
    char *cmdline = NULL;
 
148
    {
 
149
      size_t cmdline_len = 0;
 
150
      size_t cmdline_allocated = 0;
 
151
      char *tmp;
 
152
      const size_t blocksize = 1024;
 
153
      ssize_t sret;
 
154
      do {
 
155
        /* Allocate more space? */
 
156
        if(cmdline_len + blocksize + 1 > cmdline_allocated){
 
157
          tmp = realloc(cmdline, cmdline_allocated + blocksize + 1);
 
158
          if(tmp == NULL){
 
159
            error_plus(0, errno, "realloc");
 
160
            free(cmdline);
 
161
            close(cl_fd);
 
162
            return 0;
 
163
          }
 
164
          cmdline = tmp;
 
165
          cmdline_allocated += blocksize;
 
166
        }
 
167
        
 
168
        /* Read data */
 
169
        sret = read(cl_fd, cmdline + cmdline_len,
 
170
                    cmdline_allocated - cmdline_len);
 
171
        if(sret == -1){
 
172
          error_plus(0, errno, "read");
 
173
          free(cmdline);
 
174
          close(cl_fd);
 
175
          return 0;
 
176
        }
 
177
        cmdline_len += (size_t)sret;
 
178
      } while(sret != 0);
 
179
      ret = close(cl_fd);
 
180
      if(ret == -1){
 
181
        error_plus(0, errno, "close");
 
182
        free(cmdline);
 
183
        return 0;
 
184
      }
 
185
      cmdline[cmdline_len] = '\0'; /* Make sure it is terminated */
 
186
    }
 
187
    /* we now have cmdline */
 
188
    
 
189
    /* get basename */
 
190
    char *cmdline_base = strrchr(cmdline, '/');
 
191
    if(cmdline_base != NULL){
 
192
      cmdline_base += 1;                /* skip the slash */
 
193
    } else {
 
194
      cmdline_base = cmdline;
 
195
    }
 
196
    
 
197
    if(strcmp(cmdline_base, plymouth_name) != 0){
 
198
      if(debug){
 
199
        fprintf(stderr, "\"%s\" is not \"%s\"\n", cmdline_base,
 
200
                plymouth_name);
 
201
      }
 
202
      free(cmdline);
 
203
      return 0;
 
204
    }
 
205
    if(debug){
 
206
      fprintf(stderr, "\"%s\" equals \"%s\"\n", cmdline_base,
 
207
              plymouth_name);
 
208
    }
 
209
    free(cmdline);
131
210
    return 1;
132
211
  }
133
 
 
134
 
  struct dirent **direntries;
 
212
  
 
213
  struct dirent **direntries = NULL;
135
214
  int ret;
136
215
  ret = scandir("/proc", &direntries, is_plymouth, alphasort);
137
 
  if (ret == -1){
138
 
    error(1, errno, "scandir");
139
 
  }
140
 
  if (ret < 0){
141
 
    return 1;
142
 
  } else {
143
 
    return 0;
144
 
  }
 
216
  if(ret == -1){
 
217
    error_plus(1, errno, "scandir");
 
218
  }
 
219
  {
 
220
    int i = ret;
 
221
    while(i--){
 
222
      free(direntries[i]);
 
223
    }
 
224
  }
 
225
  free(direntries);
 
226
  return ret > 0;
145
227
}
146
228
 
147
229
 
175
257
      { .name = NULL }
176
258
    };
177
259
    
 
260
    __attribute__((nonnull(3)))
178
261
    error_t parse_opt (int key, char *arg, struct argp_state *state){
179
262
      errno = 0;
180
263
      switch (key){
191
274
        argp_state_help(state, state->out_stream,
192
275
                        (ARGP_HELP_STD_HELP | ARGP_HELP_EXIT_ERR)
193
276
                        & ~(unsigned int)ARGP_HELP_EXIT_OK);
 
277
        __builtin_unreachable();
194
278
      case -3:                  /* --usage */
195
279
        argp_state_help(state, state->out_stream,
196
280
                        ARGP_HELP_USAGE | ARGP_HELP_EXIT_ERR);
 
281
        __builtin_unreachable();
197
282
      case 'V':                 /* --version */
198
283
        fprintf(state->out_stream, "%s\n", argp_program_version);
199
284
        exit(argp_err_exit_status);
216
301
    case ENOMEM:
217
302
    default:
218
303
      errno = ret;
219
 
      error(0, errno, "argp_parse");
 
304
      error_plus(0, errno, "argp_parse");
220
305
      return EX_OSERR;
221
306
    case EINVAL:
222
307
      return EX_USAGE;
227
312
    fprintf(stderr, "Starting %s\n", argv[0]);
228
313
  }
229
314
 
230
 
  if (conflict_detection()){
 
315
  if(conflict_detection()){
231
316
    if(debug){
232
 
      fprintf(stderr, "Stopping %s because of conflict", argv[0]);
 
317
      fprintf(stderr, "Stopping %s because of conflict\n", argv[0]);
233
318
    }
234
319
    return EXIT_FAILURE;
235
320
  }
240
325
  
241
326
  if(tcgetattr(STDIN_FILENO, &t_old) != 0){
242
327
    int e = errno;
243
 
    error(0, errno, "tcgetattr");
 
328
    error_plus(0, errno, "tcgetattr");
244
329
    switch(e){
245
330
    case EBADF:
246
331
    case ENOTTY:
253
338
  sigemptyset(&new_action.sa_mask);
254
339
  ret = sigaddset(&new_action.sa_mask, SIGINT);
255
340
  if(ret == -1){
256
 
    error(0, errno, "sigaddset");
 
341
    error_plus(0, errno, "sigaddset");
257
342
    return EX_OSERR;
258
343
  }
259
344
  ret = sigaddset(&new_action.sa_mask, SIGHUP);
260
345
  if(ret == -1){
261
 
    error(0, errno, "sigaddset");
 
346
    error_plus(0, errno, "sigaddset");
262
347
    return EX_OSERR;
263
348
  }
264
349
  ret = sigaddset(&new_action.sa_mask, SIGTERM);
265
350
  if(ret == -1){
266
 
    error(0, errno, "sigaddset");
 
351
    error_plus(0, errno, "sigaddset");
267
352
    return EX_OSERR;
268
353
  }
269
354
  /* Need to check if the handler is SIG_IGN before handling:
272
357
  */
273
358
  ret = sigaction(SIGINT, NULL, &old_action);
274
359
  if(ret == -1){
275
 
    error(0, errno, "sigaction");
 
360
    error_plus(0, errno, "sigaction");
276
361
    return EX_OSERR;
277
362
  }
278
363
  if(old_action.sa_handler != SIG_IGN){
279
364
    ret = sigaction(SIGINT, &new_action, NULL);
280
365
    if(ret == -1){
281
 
      error(0, errno, "sigaction");
 
366
      error_plus(0, errno, "sigaction");
282
367
      return EX_OSERR;
283
368
    }
284
369
  }
285
370
  ret = sigaction(SIGHUP, NULL, &old_action);
286
371
  if(ret == -1){
287
 
    error(0, errno, "sigaction");
 
372
    error_plus(0, errno, "sigaction");
288
373
    return EX_OSERR;
289
374
  }
290
375
  if(old_action.sa_handler != SIG_IGN){
291
376
    ret = sigaction(SIGHUP, &new_action, NULL);
292
377
    if(ret == -1){
293
 
      error(0, errno, "sigaction");
 
378
      error_plus(0, errno, "sigaction");
294
379
      return EX_OSERR;
295
380
    }
296
381
  }
297
382
  ret = sigaction(SIGTERM, NULL, &old_action);
298
383
  if(ret == -1){
299
 
    error(0, errno, "sigaction");
 
384
    error_plus(0, errno, "sigaction");
300
385
    return EX_OSERR;
301
386
  }
302
387
  if(old_action.sa_handler != SIG_IGN){
303
388
    ret = sigaction(SIGTERM, &new_action, NULL);
304
389
    if(ret == -1){
305
 
      error(0, errno, "sigaction");
 
390
      error_plus(0, errno, "sigaction");
306
391
      return EX_OSERR;
307
392
    }
308
393
  }
316
401
  t_new.c_lflag &= ~(tcflag_t)ECHO;
317
402
  if(tcsetattr(STDIN_FILENO, TCSAFLUSH, &t_new) != 0){
318
403
    int e = errno;
319
 
    error(0, errno, "tcsetattr-echo");
 
404
    error_plus(0, errno, "tcsetattr-echo");
320
405
    switch(e){
321
406
    case EBADF:
322
407
    case ENOTTY:
386
471
        sret = write(STDOUT_FILENO, buffer + written, n - written);
387
472
        if(sret < 0){
388
473
          int e = errno;
389
 
          error(0, errno, "write");
 
474
          error_plus(0, errno, "write");
390
475
          switch(e){
391
476
          case EBADF:
392
477
          case EFAULT:
408
493
      sret = close(STDOUT_FILENO);
409
494
      if(sret == -1){
410
495
        int e = errno;
411
 
        error(0, errno, "close");
 
496
        error_plus(0, errno, "close");
412
497
        switch(e){
413
498
        case EBADF:
414
499
          status = EX_OSFILE;
423
508
    }
424
509
    if(sret < 0){
425
510
      int e = errno;
426
 
      if(errno != EINTR and not feof(stdin)){
427
 
        error(0, errno, "getline");
428
 
        switch(e){
429
 
        case EBADF:
430
 
          status = EX_UNAVAILABLE;
431
 
        case EIO:
432
 
        case EINVAL:
433
 
        default:
434
 
          status = EX_IOERR;
 
511
      if(errno != EINTR){
 
512
        if(not feof(stdin)){
 
513
          error_plus(0, errno, "getline");
 
514
          switch(e){
 
515
          case EBADF:
 
516
            status = EX_UNAVAILABLE;
 
517
            break;
 
518
          case EIO:
 
519
          case EINVAL:
 
520
          default:
 
521
            status = EX_IOERR;
 
522
            break;
 
523
          }
435
524
          break;
 
525
        } else {
 
526
          clearerr(stdin);
436
527
        }
437
 
        break;
438
528
      }
439
529
    }
440
 
    /* if(sret == 0), then the only sensible thing to do is to retry to
441
 
       read from stdin */
 
530
    /* if(sret == 0), then the only sensible thing to do is to retry
 
531
       to read from stdin */
442
532
    fputc('\n', stderr);
443
533
    if(debug and not quit_now){
444
534
      /* If quit_now is nonzero, we were interrupted by a signal, and
453
543
    fprintf(stderr, "Restoring terminal attributes\n");
454
544
  }
455
545
  if(tcsetattr(STDIN_FILENO, TCSAFLUSH, &t_old) != 0){
456
 
    error(0, errno, "tcsetattr+echo");
 
546
    error_plus(0, errno, "tcsetattr+echo");
457
547
  }
458
548
  
459
549
  if(quit_now){
461
551
    old_action.sa_handler = SIG_DFL;
462
552
    ret = sigaction(signal_received, &old_action, NULL);
463
553
    if(ret == -1){
464
 
      error(0, errno, "sigaction");
 
554
      error_plus(0, errno, "sigaction");
465
555
    }
466
556
    raise(signal_received);
467
557
  }