/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/askpass-fifo.c

  • Committer: Teddy Hogeborn
  • Date: 2009-10-24 19:17:52 UTC
  • Revision ID: teddy@fukt.bsnet.se-20091024191752-7g6xlf6wpp2pdexb
Convert some programs to use the exit codes from <sysexits.h>.  Change
all programs using the "argp" parsing functions to use them correctly;
checking return value, using argp_error() to report parse errors etc.

* plugin-runner.c: Use <sysexits.h> exit codes.  Always use fallback,
                   even on option errors, except for "--help", etc.
  (getplugin): Make sure "errno" is set correctly on return.
  (main): Declare our own "--help", "--usage", and "--version"
          options which do not cause the fallback to be invoked.
          In all other options, use fallback on any error.
  (parse_opt, parse_opt_config_file): Reset errno at start and return
                                      errno.  No need to check "arg"
                                      for NULL.  New "--help",
                                      "--usage", and "--version"
                                      options.
  (parse_opt): Accept empty string as global option.  Do not print
               errors which will be detected and reported later.  Do
               "argp_error()" on parse error or empty plugin names.
* plugins.d/mandos-client.c: Use <sysexits.h> exit codes.  Do not
                             return successful exit code on "--help",
                             etc. since this would give the wrong
                             message to "plugin-runner".
  (main): Declare our own "--help", "--usage", and "--version"
          options which do not return a successful exit code.
  (parse_opt): Reset errno at start and return errno.  Do
               "argp_error()" on parse errors.  New "--help",
               "--usage", and "--version" options.
* plugins.d/password-prompt.c: Use exit codes from <sysexits.h>.  Do
                               not return successful exit code on
                               "--help", etc. since this would give
                               the wrong message to "plugin-runner".
  (main): Declare our own "--help", "--usage", and "--version" options
          which do not return a successful exit code.  Do
          close(STDOUT_FILENO) after writing to check its return code.
  (parse_opt): Reset errno at start and return errno.

Show diffs side-by-side

added added

removed removed

Lines of Context:
plugins.d/askpass-fifo.c
2
2
/*
3
3
 * Askpass-FIFO - Read a password from a FIFO and output it
4
4
 * 
5
 
 * Copyright © 2008-2012 Teddy Hogeborn
6
 
 * Copyright © 2008-2012 Björn Påhlsson
 
5
 * Copyright © 2008,2009 Teddy Hogeborn
 
6
 * Copyright © 2008,2009 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@recompile.se>.
 
22
 * Contact the authors at <mandos@fukt.bsnet.se>.
23
23
 */
24
24
 
25
25
#define _GNU_SOURCE             /* TEMP_FAILURE_RETRY() */
31
31
                                   ENOENT, EEXIST, EFAULT, EMFILE,
32
32
                                   ENFILE, ENOMEM, EBADF, EINVAL, EIO,
33
33
                                   EISDIR, EFBIG */
34
 
#include <error.h>              /* error() */
35
 
#include <stdio.h>              /* fprintf(), vfprintf(),
36
 
                                   vasprintf() */
 
34
#include <stdio.h>              /* perror() */
37
35
#include <stdlib.h>             /* EXIT_FAILURE, NULL, size_t, free(),
38
36
                                   realloc(), EXIT_SUCCESS */
39
37
#include <fcntl.h>              /* open(), O_RDONLY */
41
39
                                   STDOUT_FILENO */
42
40
#include <sysexits.h>           /* EX_OSERR, EX_OSFILE,
43
41
                                   EX_UNAVAILABLE, EX_IOERR */
44
 
#include <string.h>             /* strerror() */
45
 
#include <stdarg.h>             /* va_list, va_start(), ... */
46
 
 
47
 
 
48
 
/* Function to use when printing errors */
49
 
__attribute__((format (gnu_printf, 3, 4)))
50
 
void error_plus(int status, int errnum, const char *formatstring,
51
 
                ...){
52
 
  va_list ap;
53
 
  char *text;
54
 
  int ret;
55
 
  
56
 
  va_start(ap, formatstring);
57
 
  ret = vasprintf(&text, formatstring, ap);
58
 
  if (ret == -1){
59
 
    fprintf(stderr, "Mandos plugin %s: ",
60
 
            program_invocation_short_name);
61
 
    vfprintf(stderr, formatstring, ap);
62
 
    fprintf(stderr, ": ");
63
 
    fprintf(stderr, "%s\n", strerror(errnum));
64
 
    error(status, errno, "vasprintf while printing error");
65
 
    return;
66
 
  }
67
 
  fprintf(stderr, "Mandos plugin ");
68
 
  error(status, errnum, "%s", text);
69
 
  free(text);
70
 
}
 
42
 
71
43
 
72
44
int main(__attribute__((unused))int argc,
73
45
         __attribute__((unused))char **argv){
79
51
  ret = mkfifo(passfifo, S_IRUSR | S_IWUSR);
80
52
  if(ret == -1){
81
53
    int e = errno;
 
54
    perror("mkfifo");
82
55
    switch(e){
83
56
    case EACCES:
84
57
    case ENOTDIR:
85
58
    case ELOOP:
86
 
      error_plus(EX_OSFILE, errno, "mkfifo");
 
59
      return EX_OSFILE;
87
60
    case ENAMETOOLONG:
88
61
    case ENOSPC:
89
62
    case EROFS:
90
63
    default:
91
 
      error_plus(EX_OSERR, errno, "mkfifo");
 
64
      return EX_OSERR;
92
65
    case ENOENT:
93
 
      /* no "/lib/cryptsetup"? */
94
 
      error_plus(EX_UNAVAILABLE, errno, "mkfifo");
 
66
      return EX_UNAVAILABLE;    /* no "/lib/cryptsetup"? */
95
67
    case EEXIST:
96
68
      break;                    /* not an error */
97
69
    }
101
73
  int fifo_fd = open(passfifo, O_RDONLY);
102
74
  if(fifo_fd == -1){
103
75
    int e = errno;
104
 
    error_plus(0, errno, "open");
 
76
    perror("open");
105
77
    switch(e){
106
78
    case EACCES:
107
79
    case ENOENT:
129
101
      if(buf_len + blocksize > buf_allocated){
130
102
        char *tmp = realloc(buf, buf_allocated + blocksize);
131
103
        if(tmp == NULL){
132
 
          error_plus(0, errno, "realloc");
 
104
          perror("realloc");
133
105
          free(buf);
134
106
          return EX_OSERR;
135
107
        }
141
113
        int e = errno;
142
114
        free(buf);
143
115
        errno = e;
144
 
        error_plus(0, errno, "read");
 
116
        perror("read");
145
117
        switch(e){
146
118
        case EBADF:
147
119
        case EFAULT:
169
141
      int e = errno;
170
142
      free(buf);
171
143
      errno = e;
172
 
      error_plus(0, errno, "write");
 
144
      perror("write");
173
145
      switch(e){
174
146
      case EBADF:
175
147
      case EFAULT:
189
161
  ret = close(STDOUT_FILENO);
190
162
  if(ret == -1){
191
163
    int e = errno;
192
 
    error_plus(0, errno, "close");
 
164
    perror("close");
193
165
    switch(e){
194
166
    case EBADF:
195
167
      return EX_OSFILE;