/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 at bsnet
  • Date: 2011-12-25 00:40:09 UTC
  • Revision ID: teddy@fukt.bsnet.se-20111225004009-n5uimmac6h8djtv8
* plugin-runner.c (add_to_char_array): Added "nonnull" attribute.
  (add_argument): Added "nonnull" attribute on the "arg" argument.
  (add_environment): Added "nonnull" attribute on the "def" argument.
  (print_out_password, free_plugin): Added "nonnull" attribute.
  (main/parse_opt): Added "nonnull" attribute on the "state" argument.
* plugins.d/mandos-client.c (perror_plus): Bug fix; restore errno
                                           after fprintf().
* plugins.d/password-prompt.c (fprintf_plus): New.
 (conflict_detection/is_plymouth, main/parse_opt): Added "nonnull"
                                                   attribute.
 (conflict_detection/is_plymouth, conflict_detection, main): Bug fix;
                                                             Call
                                                             error_plus()
                                                             instead
                                                             of
                                                             error().
  (main/parse_opt): Added "nonnull" attribute on the "state" argument.
* plugins.d/plymouth.c (exec_and_wait): Added "nonnull" attribute on
                                        the "path" and "argv"
                                        arguments.
  (is_plymouth): Added "nonnull" attribute.

Show diffs side-by-side

added added

removed removed

Lines of Context:
plugins.d/password-prompt.c
72
72
/* Needed for conflict resolution */
73
73
const char plymouth_name[] = "plymouthd";
74
74
 
 
75
__attribute__((format (gnu_printf, 2, 3), nonnull(1)))
 
76
int fprintf_plus(FILE *stream, const char *format, ...){
 
77
  va_list ap;
 
78
  va_start (ap, format);
 
79
  
 
80
  TEMP_FAILURE_RETRY(fprintf(stream, "Mandos plugin %s: ",
 
81
                             program_invocation_short_name));
 
82
  return TEMP_FAILURE_RETRY(vfprintf(stream, format, ap));
 
83
}
 
84
 
75
85
/* Function to use when printing errors */
76
86
__attribute__((format (gnu_printf, 3, 4)))
77
87
void error_plus(int status, int errnum, const char *formatstring,
86
96
    fprintf(stderr, "Mandos plugin %s: ",
87
97
            program_invocation_short_name);
88
98
    vfprintf(stderr, formatstring, ap);
89
 
    fprintf(stderr, ": ");
90
 
    fprintf(stderr, "%s\n", strerror(errnum));
 
99
    fprintf(stderr, ": %s\n", strerror(errnum));
91
100
    error(status, errno, "vasprintf while printing error");
92
101
    return;
93
102
  }
110
119
     from the terminal.  Password-prompt will exit if it detects
111
120
     plymouth since plymouth performs the same functionality.
112
121
   */
 
122
  __attribute__((nonnull))
113
123
  int is_plymouth(const struct dirent *proc_entry){
114
124
    int ret;
115
125
    int cl_fd;
129
139
    ret = asprintf(&cmdline_filename, "/proc/%s/cmdline",
130
140
                   proc_entry->d_name);
131
141
    if(ret == -1){
132
 
      error(0, errno, "asprintf");
 
142
      error_plus(0, errno, "asprintf");
133
143
      return 0;
134
144
    }
135
145
    
138
148
    free(cmdline_filename);
139
149
    if(cl_fd == -1){
140
150
      if(errno != ENOENT){
141
 
        error(0, errno, "open");
 
151
        error_plus(0, errno, "open");
142
152
      }
143
153
      return 0;
144
154
    }
155
165
        if(cmdline_len + blocksize + 1 > cmdline_allocated){
156
166
          tmp = realloc(cmdline, cmdline_allocated + blocksize + 1);
157
167
          if(tmp == NULL){
158
 
            error(0, errno, "realloc");
 
168
            error_plus(0, errno, "realloc");
159
169
            free(cmdline);
160
170
            close(cl_fd);
161
171
            return 0;
168
178
        sret = read(cl_fd, cmdline + cmdline_len,
169
179
                    cmdline_allocated - cmdline_len);
170
180
        if(sret == -1){
171
 
          error(0, errno, "read");
 
181
          error_plus(0, errno, "read");
172
182
          free(cmdline);
173
183
          close(cl_fd);
174
184
          return 0;
177
187
      } while(sret != 0);
178
188
      ret = close(cl_fd);
179
189
      if(ret == -1){
180
 
        error(0, errno, "close");
 
190
        error_plus(0, errno, "close");
181
191
        free(cmdline);
182
192
        return 0;
183
193
      }
213
223
  int ret;
214
224
  ret = scandir("/proc", &direntries, is_plymouth, alphasort);
215
225
  if (ret == -1){
216
 
    error(1, errno, "scandir");
 
226
    error_plus(1, errno, "scandir");
217
227
  }
218
228
  free(direntries);
219
229
  return ret > 0;
250
260
      { .name = NULL }
251
261
    };
252
262
    
 
263
    __attribute__((nonnull(3)))
253
264
    error_t parse_opt (int key, char *arg, struct argp_state *state){
254
265
      errno = 0;
255
266
      switch (key){
291
302
    case ENOMEM:
292
303
    default:
293
304
      errno = ret;
294
 
      error(0, errno, "argp_parse");
 
305
      error_plus(0, errno, "argp_parse");
295
306
      return EX_OSERR;
296
307
    case EINVAL:
297
308
      return EX_USAGE;
315
326
  
316
327
  if(tcgetattr(STDIN_FILENO, &t_old) != 0){
317
328
    int e = errno;
318
 
    error(0, errno, "tcgetattr");
 
329
    error_plus(0, errno, "tcgetattr");
319
330
    switch(e){
320
331
    case EBADF:
321
332
    case ENOTTY:
328
339
  sigemptyset(&new_action.sa_mask);
329
340
  ret = sigaddset(&new_action.sa_mask, SIGINT);
330
341
  if(ret == -1){
331
 
    error(0, errno, "sigaddset");
 
342
    error_plus(0, errno, "sigaddset");
332
343
    return EX_OSERR;
333
344
  }
334
345
  ret = sigaddset(&new_action.sa_mask, SIGHUP);
335
346
  if(ret == -1){
336
 
    error(0, errno, "sigaddset");
 
347
    error_plus(0, errno, "sigaddset");
337
348
    return EX_OSERR;
338
349
  }
339
350
  ret = sigaddset(&new_action.sa_mask, SIGTERM);
340
351
  if(ret == -1){
341
 
    error(0, errno, "sigaddset");
 
352
    error_plus(0, errno, "sigaddset");
342
353
    return EX_OSERR;
343
354
  }
344
355
  /* Need to check if the handler is SIG_IGN before handling:
347
358
  */
348
359
  ret = sigaction(SIGINT, NULL, &old_action);
349
360
  if(ret == -1){
350
 
    error(0, errno, "sigaction");
 
361
    error_plus(0, errno, "sigaction");
351
362
    return EX_OSERR;
352
363
  }
353
364
  if(old_action.sa_handler != SIG_IGN){
354
365
    ret = sigaction(SIGINT, &new_action, NULL);
355
366
    if(ret == -1){
356
 
      error(0, errno, "sigaction");
 
367
      error_plus(0, errno, "sigaction");
357
368
      return EX_OSERR;
358
369
    }
359
370
  }
360
371
  ret = sigaction(SIGHUP, NULL, &old_action);
361
372
  if(ret == -1){
362
 
    error(0, errno, "sigaction");
 
373
    error_plus(0, errno, "sigaction");
363
374
    return EX_OSERR;
364
375
  }
365
376
  if(old_action.sa_handler != SIG_IGN){
366
377
    ret = sigaction(SIGHUP, &new_action, NULL);
367
378
    if(ret == -1){
368
 
      error(0, errno, "sigaction");
 
379
      error_plus(0, errno, "sigaction");
369
380
      return EX_OSERR;
370
381
    }
371
382
  }
372
383
  ret = sigaction(SIGTERM, NULL, &old_action);
373
384
  if(ret == -1){
374
 
    error(0, errno, "sigaction");
 
385
    error_plus(0, errno, "sigaction");
375
386
    return EX_OSERR;
376
387
  }
377
388
  if(old_action.sa_handler != SIG_IGN){
378
389
    ret = sigaction(SIGTERM, &new_action, NULL);
379
390
    if(ret == -1){
380
 
      error(0, errno, "sigaction");
 
391
      error_plus(0, errno, "sigaction");
381
392
      return EX_OSERR;
382
393
    }
383
394
  }
391
402
  t_new.c_lflag &= ~(tcflag_t)ECHO;
392
403
  if(tcsetattr(STDIN_FILENO, TCSAFLUSH, &t_new) != 0){
393
404
    int e = errno;
394
 
    error(0, errno, "tcsetattr-echo");
 
405
    error_plus(0, errno, "tcsetattr-echo");
395
406
    switch(e){
396
407
    case EBADF:
397
408
    case ENOTTY:
461
472
        sret = write(STDOUT_FILENO, buffer + written, n - written);
462
473
        if(sret < 0){
463
474
          int e = errno;
464
 
          error(0, errno, "write");
 
475
          error_plus(0, errno, "write");
465
476
          switch(e){
466
477
          case EBADF:
467
478
          case EFAULT:
483
494
      sret = close(STDOUT_FILENO);
484
495
      if(sret == -1){
485
496
        int e = errno;
486
 
        error(0, errno, "close");
 
497
        error_plus(0, errno, "close");
487
498
        switch(e){
488
499
        case EBADF:
489
500
          status = EX_OSFILE;
499
510
    if(sret < 0){
500
511
      int e = errno;
501
512
      if(errno != EINTR and not feof(stdin)){
502
 
        error(0, errno, "getline");
 
513
        error_plus(0, errno, "getline");
503
514
        switch(e){
504
515
        case EBADF:
505
516
          status = EX_UNAVAILABLE;
528
539
    fprintf(stderr, "Restoring terminal attributes\n");
529
540
  }
530
541
  if(tcsetattr(STDIN_FILENO, TCSAFLUSH, &t_old) != 0){
531
 
    error(0, errno, "tcsetattr+echo");
 
542
    error_plus(0, errno, "tcsetattr+echo");
532
543
  }
533
544
  
534
545
  if(quit_now){
536
547
    old_action.sa_handler = SIG_DFL;
537
548
    ret = sigaction(signal_received, &old_action, NULL);
538
549
    if(ret == -1){
539
 
      error(0, errno, "sigaction");
 
550
      error_plus(0, errno, "sigaction");
540
551
    }
541
552
    raise(signal_received);
542
553
  }