/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 plugin-runner.c

  • Committer: Teddy Hogeborn
  • Date: 2014-03-23 22:12:57 UTC
  • Revision ID: teddy@recompile.se-20140323221257-9lkufevfdn4kwoq6
Use openat() etc. in plugin-runner.  Also one less useless warning.

* plugin-runner.c (main): Only print warning about failure to work
                          around Debian bug #633582 if the plugin
                          directory actually exists.  Use openat(),
                          fstat(), faccessat(), and eliminate
                          asprintf().

Show diffs side-by-side

added added

removed removed

Lines of Context:
plugin-runner.c
23
23
 */
24
24
 
25
25
#define _GNU_SOURCE             /* TEMP_FAILURE_RETRY(), getline(),
26
 
                                   asprintf(), O_CLOEXEC */
 
26
                                   O_CLOEXEC */
27
27
#include <stddef.h>             /* size_t, NULL */
28
28
#include <stdlib.h>             /* malloc(), exit(), EXIT_SUCCESS,
29
29
                                   realloc() */
30
30
#include <stdbool.h>            /* bool, true, false */
31
31
#include <stdio.h>              /* fileno(), fprintf(),
32
 
                                   stderr, STDOUT_FILENO */
33
 
#include <sys/types.h>          /* DIR, fdopendir(), stat(), struct
 
32
                                   stderr, STDOUT_FILENO, fclose() */
 
33
#include <sys/types.h>          /* DIR, fdopendir(), fstat(), struct
34
34
                                   stat, waitpid(), WIFEXITED(),
35
35
                                   WEXITSTATUS(), wait(), pid_t,
36
36
                                   uid_t, gid_t, getuid(), getgid(),
40
40
#include <sys/wait.h>           /* wait(), waitpid(), WIFEXITED(),
41
41
                                   WEXITSTATUS(), WTERMSIG(),
42
42
                                   WCOREDUMP() */
43
 
#include <sys/stat.h>           /* struct stat, stat(), S_ISREG() */
 
43
#include <sys/stat.h>           /* struct stat, fstat(), S_ISREG() */
44
44
#include <iso646.h>             /* and, or, not */
45
45
#include <dirent.h>             /* DIR, struct dirent, fdopendir(),
46
46
                                   readdir(), closedir(), dirfd() */
47
 
#include <unistd.h>             /* struct stat, stat(), S_ISREG(),
48
 
                                   fcntl(), setuid(), setgid(),
49
 
                                   F_GETFD, F_SETFD, FD_CLOEXEC,
50
 
                                   access(), pipe(), fork(), close()
51
 
                                   dup2(), STDOUT_FILENO, _exit(),
52
 
                                   execv(), write(), read(),
53
 
                                   close() */
 
47
#include <unistd.h>             /* fcntl(), F_GETFD, F_SETFD,
 
48
                                   FD_CLOEXEC, write(), STDOUT_FILENO,
 
49
                                   struct stat, fstat(), close(),
 
50
                                   setgid(), setuid(), S_ISREG(),
 
51
                                   faccessat() pipe(), fork(),
 
52
                                   _exit(), dup2(), fexecve(), read()
 
53
                                */
54
54
#include <fcntl.h>              /* fcntl(), F_GETFD, F_SETFD,
55
 
                                   FD_CLOEXEC */
56
 
#include <string.h>             /* strsep, strlen(), asprintf(),
57
 
                                   strsignal(), strcmp(), strncmp() */
 
55
                                   FD_CLOEXEC, openat() */
 
56
#include <string.h>             /* strsep, strlen(), strsignal(),
 
57
                                   strcmp(), strncmp() */
58
58
#include <errno.h>              /* errno */
59
59
#include <argp.h>               /* struct argp_option, struct
60
60
                                   argp_state, struct argp,
105
105
 
106
106
/* Gets an existing plugin based on name,
107
107
   or if none is found, creates a new one */
 
108
__attribute__((warn_unused_result))
108
109
static plugin *getplugin(char *name){
109
110
  /* Check for existing plugin with that name */
110
111
  for(plugin *p = plugin_list; p != NULL; p = p->next){
171
172
}
172
173
 
173
174
/* Helper function for add_argument and add_environment */
174
 
__attribute__((nonnull))
 
175
__attribute__((nonnull, warn_unused_result))
175
176
static bool add_to_char_array(const char *new, char ***array,
176
177
                              int *len){
177
178
  /* Resize the pointed-to array to hold one more pointer */
202
203
}
203
204
 
204
205
/* Add to a plugin's argument vector */
205
 
__attribute__((nonnull(2)))
 
206
__attribute__((nonnull(2), warn_unused_result))
206
207
static bool add_argument(plugin *p, const char *arg){
207
208
  if(p == NULL){
208
209
    return false;
211
212
}
212
213
 
213
214
/* Add to a plugin's environment */
214
 
__attribute__((nonnull(2)))
 
215
__attribute__((nonnull(2), warn_unused_result))
215
216
static bool add_environment(plugin *p, const char *def, bool replace){
216
217
  if(p == NULL){
217
218
    return false;
244
245
 * Descriptor Flags".
245
246
 | [[info:libc:Descriptor%20Flags][File Descriptor Flags]] |
246
247
 */
 
248
__attribute__((warn_unused_result))
247
249
static int set_cloexec_flag(int fd){
248
250
  int ret = (int)TEMP_FAILURE_RETRY(fcntl(fd, F_GETFD, 0));
249
251
  /* If reading the flags failed, return error indication now. */
291
293
}
292
294
 
293
295
/* Prints out a password to stdout */
294
 
__attribute__((nonnull))
 
296
__attribute__((nonnull, warn_unused_result))
295
297
static bool print_out_password(const char *buffer, size_t length){
296
298
  ssize_t ret;
297
299
  for(size_t written = 0; written < length; written += (size_t)ret){
361
363
                                      .sa_flags = SA_NOCLDSTOP };
362
364
  char **custom_argv = NULL;
363
365
  int custom_argc = 0;
 
366
  int dir_fd;
364
367
  
365
368
  /* Establish a signal handler */
366
369
  sigemptyset(&sigchld_action.sa_mask);
437
440
            break;
438
441
          }
439
442
        }
 
443
        errno = 0;
440
444
      }
441
445
      break;
442
446
    case 'G':                   /* --global-env */
443
 
      add_environment(getplugin(NULL), arg, true);
 
447
      if(add_environment(getplugin(NULL), arg, true)){
 
448
        errno = 0;
 
449
      }
444
450
      break;
445
451
    case 'o':                   /* --options-for */
446
452
      {
463
469
            break;
464
470
          }
465
471
        }
 
472
        errno = 0;
466
473
      }
467
474
      break;
468
475
    case 'E':                   /* --env-for */
480
487
          errno = EINVAL;
481
488
          break;
482
489
        }
483
 
        add_environment(getplugin(arg), envdef, true);
 
490
        if(add_environment(getplugin(arg), envdef, true)){
 
491
          errno = 0;
 
492
        }
484
493
      }
485
494
      break;
486
495
    case 'd':                   /* --disable */
488
497
        plugin *p = getplugin(arg);
489
498
        if(p != NULL){
490
499
          p->disabled = true;
 
500
          errno = 0;
491
501
        }
492
502
      }
493
503
      break;
496
506
        plugin *p = getplugin(arg);
497
507
        if(p != NULL){
498
508
          p->disabled = false;
 
509
          errno = 0;
499
510
        }
500
511
      }
501
512
      break;
502
513
    case 128:                   /* --plugin-dir */
503
514
      free(plugindir);
504
515
      plugindir = strdup(arg);
 
516
      if(plugindir != NULL){
 
517
        errno = 0;
 
518
      }
505
519
      break;
506
520
    case 129:                   /* --config-file */
507
521
      /* This is already done by parse_opt_config_file() */
515
529
        break;
516
530
      }
517
531
      uid = (uid_t)tmp_id;
 
532
      errno = 0;
518
533
      break;
519
534
    case 131:                   /* --groupid */
520
535
      tmp_id = strtoimax(arg, &tmp, 10);
525
540
        break;
526
541
      }
527
542
      gid = (gid_t)tmp_id;
 
543
      errno = 0;
528
544
      break;
529
545
    case 132:                   /* --debug */
530
546
      debug = true;
578
594
    case 129:                   /* --config-file */
579
595
      free(argfile);
580
596
      argfile = strdup(arg);
 
597
      if(argfile != NULL){
 
598
        errno = 0;
 
599
      }
581
600
      break;
582
601
    case 130:                   /* --userid */
583
602
    case 131:                   /* --groupid */
761
780
       <http://bugs.debian.org/633582> */
762
781
    int plugindir_fd = open(/* plugindir or */ PDIR, O_RDONLY);
763
782
    if(plugindir_fd == -1){
764
 
      error(0, errno, "open");
 
783
      if(errno != ENOENT){
 
784
        error(0, errno, "open(\"" PDIR "\")");
 
785
      }
765
786
    } else {
766
787
      ret = (int)TEMP_FAILURE_RETRY(fstat(plugindir_fd, &st));
767
788
      if(ret == -1){
790
811
  
791
812
  /* Open plugin directory with close_on_exec flag */
792
813
  {
793
 
    int dir_fd = -1;
794
 
    if(plugindir == NULL){
795
 
      dir_fd = open(PDIR, O_RDONLY |
796
 
#ifdef O_CLOEXEC
797
 
                    O_CLOEXEC
798
 
#else  /* not O_CLOEXEC */
799
 
                    0
800
 
#endif  /* not O_CLOEXEC */
801
 
                    );
802
 
    } else {
803
 
      dir_fd = open(plugindir, O_RDONLY |
804
 
#ifdef O_CLOEXEC
805
 
                    O_CLOEXEC
806
 
#else  /* not O_CLOEXEC */
807
 
                    0
808
 
#endif  /* not O_CLOEXEC */
809
 
                    );
810
 
    }
 
814
    dir_fd = open(plugindir != NULL ? plugindir : PDIR, O_RDONLY |
 
815
#ifdef O_CLOEXEC
 
816
                  O_CLOEXEC
 
817
#else  /* not O_CLOEXEC */
 
818
                  0
 
819
#endif  /* not O_CLOEXEC */
 
820
                  );
811
821
    if(dir_fd == -1){
812
822
      error(0, errno, "Could not open plugin dir");
813
823
      exitstatus = EX_UNAVAILABLE;
914
924
      }
915
925
    }
916
926
    
917
 
    char *filename;
918
 
    if(plugindir == NULL){
919
 
      ret = (int)TEMP_FAILURE_RETRY(asprintf(&filename, PDIR "/%s",
920
 
                                             dirst->d_name));
921
 
    } else {
922
 
      ret = (int)TEMP_FAILURE_RETRY(asprintf(&filename, "%s/%s",
923
 
                                             plugindir,
924
 
                                             dirst->d_name));
 
927
    int plugin_fd = openat(dir_fd, dirst->d_name, O_RDONLY |
 
928
#ifdef O_CLOEXEC
 
929
                            O_CLOEXEC
 
930
#else  /* not O_CLOEXEC */
 
931
                            0
 
932
#endif  /* not O_CLOEXEC */
 
933
                            );
 
934
    if(plugin_fd == -1){
 
935
      error(0, errno, "Could not open plugin");
 
936
      continue;
925
937
    }
 
938
#ifndef O_CLOEXEC
 
939
  /* Set the FD_CLOEXEC flag on the plugin FD */
 
940
    ret = set_cloexec_flag(plugin_fd);
926
941
    if(ret < 0){
927
 
      error(0, errno, "asprintf");
 
942
      error(0, errno, "set_cloexec_flag");
 
943
      TEMP_FAILURE_RETRY(close(plugin_fd));
928
944
      continue;
929
945
    }
930
 
    
931
 
    ret = (int)TEMP_FAILURE_RETRY(stat(filename, &st));
 
946
#endif  /* O_CLOEXEC */
 
947
    ret = (int)TEMP_FAILURE_RETRY(fstat(plugin_fd, &st));
932
948
    if(ret == -1){
933
949
      error(0, errno, "stat");
934
 
      free(filename);
 
950
      TEMP_FAILURE_RETRY(close(plugin_fd));
935
951
      continue;
936
952
    }
937
953
    
938
954
    /* Ignore non-executable files */
939
955
    if(not S_ISREG(st.st_mode)
940
 
       or (TEMP_FAILURE_RETRY(access(filename, X_OK)) != 0)){
 
956
       or (TEMP_FAILURE_RETRY(faccessat(dir_fd, dirst->d_name, X_OK,
 
957
                                        0)) != 0)){
941
958
      if(debug){
942
 
        fprintf(stderr, "Ignoring plugin dir entry \"%s\""
943
 
                " with bad type or mode\n", filename);
 
959
        fprintf(stderr, "Ignoring plugin dir entry \"%s/%s\""
 
960
                " with bad type or mode\n",
 
961
                plugindir != NULL ? plugindir : PDIR, dirst->d_name);
944
962
      }
945
 
      free(filename);
 
963
      TEMP_FAILURE_RETRY(close(plugin_fd));
946
964
      continue;
947
965
    }
948
966
    
949
967
    plugin *p = getplugin(dirst->d_name);
950
968
    if(p == NULL){
951
969
      error(0, errno, "getplugin");
952
 
      free(filename);
 
970
      TEMP_FAILURE_RETRY(close(plugin_fd));
953
971
      continue;
954
972
    }
955
973
    if(p->disabled){
957
975
        fprintf(stderr, "Ignoring disabled plugin \"%s\"\n",
958
976
                dirst->d_name);
959
977
      }
960
 
      free(filename);
 
978
      TEMP_FAILURE_RETRY(close(plugin_fd));
961
979
      continue;
962
980
    }
963
981
    {
977
995
        }
978
996
      }
979
997
    }
980
 
    /* If this plugin has any environment variables, we will call
981
 
       using execve and need to duplicate the environment from this
982
 
       process, too. */
 
998
    /* If this plugin has any environment variables, we need to
 
999
       duplicate the environment from this process, too. */
983
1000
    if(p->environ[0] != NULL){
984
1001
      for(char **e = environ; *e != NULL; e++){
985
1002
        if(not add_environment(p, *e, false)){
1051
1068
           above and must now close it manually here. */
1052
1069
        closedir(dir);
1053
1070
      }
1054
 
      if(p->environ[0] == NULL){
1055
 
        if(execv(filename, p->argv) < 0){
1056
 
          error(0, errno, "execv for %s", filename);
1057
 
          _exit(EX_OSERR);
1058
 
        }
1059
 
      } else {
1060
 
        if(execve(filename, p->argv, p->environ) < 0){
1061
 
          error(0, errno, "execve for %s", filename);
1062
 
          _exit(EX_OSERR);
1063
 
        }
 
1071
      if(fexecve(plugin_fd, p->argv,
 
1072
                (p->environ[0] != NULL) ? p->environ : environ) < 0){
 
1073
        error(0, errno, "fexecve for %s/%s",
 
1074
              plugindir != NULL ? plugindir : PDIR, dirst->d_name);
 
1075
        _exit(EX_OSERR);
1064
1076
      }
1065
1077
      /* no return */
1066
1078
    }
1067
1079
    /* Parent process */
1068
1080
    TEMP_FAILURE_RETRY(close(pipefd[1])); /* Close unused write end of
1069
1081
                                             pipe */
1070
 
    free(filename);
 
1082
    TEMP_FAILURE_RETRY(close(plugin_fd));
1071
1083
    plugin *new_plugin = getplugin(dirst->d_name);
1072
1084
    if(new_plugin == NULL){
1073
1085
      error(0, errno, "getplugin");