/mandos/release

To get this branch, use:
bzr branch http://bzr.recompile.se/loggerhead/mandos/release

« back to all changes in this revision

Viewing changes to plugin-runner.c

Four new interrelated features:

1. Support using a different network interface via both initramfs.conf
   (the DEVICE setting) and the kernel command line (sixth field of
   the "ip=" option as in Linux' Documentation/nfsroot.txt).

2. Support connecting to a specified Mandos server directly using a
   kernel command line option ("mandos=connect:<ADDRESS>:<PORT>").

3. Support connecting directly to an IPv4 address (and port) using the
   "--connect" option of mandos-client.

4. Support an empty string to the --interface option to mandos-client.

* Makefile (WARN): Increase strictness by changing to
                   "-Wstrict-aliasing=1".

* debian/mandos-client.README.Debian (Use the Correct Network
  Interface): Changed to refer to initramfs.conf and nfsroot.txt.
  (Test the Server): Improve wording.
  (Non-local Connection): New section.
* initramfs-tools-script: Obey DEVICE environment variable and setting
                          from "/conf/initramfs.conf".  Also let any
                          "ip=" kernel command line option override
                          it.  Support new "mandos=connect" option.
                          Call "configure_networking" to set up IP
                          address on interface if necessary.
* plugin-runner.conf: Change example.
* plugins.d/mandos-client.c: Some whitespace and comment changes.
  (start_mandos_communication): Take an additional argument for
                                address family, all callers changed.
                                Connect to an IPv4 address if address
                                family is AF_INET.  Only set IPv6
                                scope_id for link-local addresses.
  (main): Accept empty interface name; this will not bring up any
         interface and leave the interface as unspecified.  Also do
         not restore kernel log level if lowering it failed.
* plugins.d/mandos-client.xml (OPTIONS): Document that the
                                         "--interface" option accepts
                                         an empty string.
  (EXAMPLE): Change example IPv6 address to a link-local address.

Show diffs side-by-side

added added

removed removed

Lines of Context:
plugin-runner.c
89
89
  size_t buffer_size;
90
90
  size_t buffer_length;
91
91
  bool eof;
92
 
  volatile bool completed;
93
 
  volatile int status;
 
92
  volatile sig_atomic_t completed;
 
93
  int status;
94
94
  struct plugin *next;
95
95
} plugin;
96
96
 
115
115
  if(name != NULL){
116
116
    copy_name = strdup(name);
117
117
    if(copy_name == NULL){
 
118
      free(new_plugin);
118
119
      return NULL;
119
120
    }
120
121
  }
121
122
  
122
 
  *new_plugin = (plugin) { .name = copy_name,
123
 
                           .argc = 1,
124
 
                           .disabled = false,
125
 
                           .next = plugin_list };
 
123
  *new_plugin = (plugin){ .name = copy_name,
 
124
                          .argc = 1,
 
125
                          .disabled = false,
 
126
                          .next = plugin_list };
126
127
  
127
128
  new_plugin->argv = malloc(sizeof(char *) * 2);
128
129
  if(new_plugin->argv == NULL){
222
223
/* Mark processes as completed when they exit, and save their exit
223
224
   status. */
224
225
static void handle_sigchld(__attribute__((unused)) int sig){
 
226
  int old_errno = errno;
225
227
  while(true){
226
228
    plugin *proc = plugin_list;
227
229
    int status;
231
233
      break;
232
234
    }
233
235
    if(pid == -1){
234
 
      if(errno != ECHILD){
235
 
        perror("waitpid");
 
236
      if(errno == ECHILD){
 
237
        /* No child processes */
 
238
        break;
236
239
      }
237
 
      /* No child processes */
238
 
      break;
 
240
      perror("waitpid");
239
241
    }
240
242
    
241
243
    /* A child exited, find it in process_list */
247
249
      continue;
248
250
    }
249
251
    proc->status = status;
250
 
    proc->completed = true;
 
252
    proc->completed = 1;
251
253
  }
 
254
  errno = old_errno;
252
255
}
253
256
 
254
257
/* Prints out a password to stdout */
375
378
  };
376
379
  
377
380
  error_t parse_opt(int key, char *arg, __attribute__((unused))
378
 
                    struct argp_state *state) {
379
 
    switch(key) {
 
381
                    struct argp_state *state){
 
382
    switch(key){
380
383
    case 'g':                   /* --global-options */
381
384
      if(arg != NULL){
382
385
        char *p;
511
514
     ignores everything but the --config-file option. */
512
515
  error_t parse_opt_config_file(int key, char *arg,
513
516
                                __attribute__((unused))
514
 
                                struct argp_state *state) {
515
 
    switch(key) {
 
517
                                struct argp_state *state){
 
518
    switch(key){
516
519
    case 'g':                   /* --global-options */
517
520
    case 'G':                   /* --global-env */
518
521
    case 'o':                   /* --options-for */
661
664
  }
662
665
  
663
666
  /* Strip permissions down to nobody */
 
667
  setgid(gid);
 
668
  if(ret == -1){
 
669
    perror("setgid");
 
670
  }
664
671
  ret = setuid(uid);
665
672
  if(ret == -1){
666
673
    perror("setuid");
667
 
  }  
668
 
  setgid(gid);
669
 
  if(ret == -1){
670
 
    perror("setgid");
671
674
  }
672
675
  
673
676
  if(plugindir == NULL){
958
961
       from one of them */
959
962
    for(plugin *proc = plugin_list; proc != NULL;){
960
963
      /* Is this process completely done? */
961
 
      if(proc->eof and proc->completed){
 
964
      if(proc->completed and proc->eof){
962
965
        /* Only accept the plugin output if it exited cleanly */
963
966
        if(not WIFEXITED(proc->status)
964
967
           or WEXITSTATUS(proc->status) != 0){
966
969
 
967
970
          if(debug){
968
971
            if(WIFEXITED(proc->status)){
969
 
              fprintf(stderr, "Plugin %" PRIdMAX " exited with status"
970
 
                      " %d\n", (intmax_t) (proc->pid),
 
972
              fprintf(stderr, "Plugin %s [%" PRIdMAX "] exited with"
 
973
                      " status %d\n", proc->name,
 
974
                      (intmax_t) (proc->pid),
971
975
                      WEXITSTATUS(proc->status));
972
 
            } else if(WIFSIGNALED(proc->status)) {
973
 
              fprintf(stderr, "Plugin %" PRIdMAX " killed by signal"
974
 
                      " %d\n", (intmax_t) (proc->pid),
 
976
            } else if(WIFSIGNALED(proc->status)){
 
977
              fprintf(stderr, "Plugin %s [%" PRIdMAX "] killed by"
 
978
                      " signal %d\n", proc->name,
 
979
                      (intmax_t) (proc->pid),
975
980
                      WTERMSIG(proc->status));
976
981
            } else if(WCOREDUMP(proc->status)){
977
 
              fprintf(stderr, "Plugin %" PRIdMAX " dumped core\n",
978
 
                      (intmax_t) (proc->pid));
 
982
              fprintf(stderr, "Plugin %s [%" PRIdMAX "] dumped"
 
983
                      " core\n", proc->name, (intmax_t) (proc->pid));
979
984
            }
980
985
          }
981
986