/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 plugbasedclient.c

  • Committer: Teddy Hogeborn
  • Date: 2008-08-01 20:03:03 UTC
  • Revision ID: teddy@fukt.bsnet.se-20080801200303-3xjn9gjewg87365i
* plugbasedclient.c (main): Check if plugin dir could be opened.  Set
                            FD_CLOEXEC on the directory, if possible,
                            and also on both ends of the pipe.  Do not
                            allocate buffer for process in advance.
                            Free the plugin list.  Bug fix: exit if no
                            plugins found, not if any found.  Set
                            "exitstatus" in many places.  Simplify and
                            un-indent the loop reading process pipes.
                            Renamed process list iterator to "proc".
                            Bug fix:  retry password write on EINTR.
                            Free the process list properly, including
                            the process structs themselves.

Show diffs side-by-side

added added

removed removed

Lines of Context:
plugbasedclient.c
50
50
                                   struct argp_state, struct argp,
51
51
                                   argp_parse() */
52
52
 
53
 
#define BUFFER_SIZE 256
54
 
 
55
53
struct process;
56
54
 
57
55
typedef struct process{
60
58
  char *buffer;
61
59
  size_t buffer_size;
62
60
  size_t buffer_length;
63
 
  bool eof;
64
 
  bool completed;
65
 
  int status;
66
61
  struct process *next;
67
62
} process;
68
63
 
74
69
  struct plugin *next;
75
70
} plugin;
76
71
 
77
 
static plugin *getplugin(char *name, plugin **plugin_list){
 
72
plugin *getplugin(char *name, plugin **plugin_list){
78
73
  for (plugin *p = *plugin_list; p != NULL; p = p->next){
79
74
    if ((p->name == name)
80
75
        or (p->name and name and (strcmp(p->name, name) == 0))){
103
98
  return new_plugin;
104
99
}
105
100
 
106
 
static void addargument(plugin *p, char *arg){
 
101
void addargument(plugin *p, char *arg){
107
102
  p->argv[p->argc] = arg;
108
103
  p->argv = realloc(p->argv, sizeof(char *) * (size_t)(p->argc + 2));
109
104
  if (p->argv == NULL){
119
114
 * Descriptor Flags".
120
115
 * *Note File Descriptor Flags:(libc)Descriptor Flags.
121
116
 */
122
 
static int set_cloexec_flag(int fd)
 
117
int set_cloexec_flag(int fd)
123
118
{
124
119
  int ret = fcntl(fd, F_GETFD, 0);
125
120
  /* If reading the flags failed, return error indication now. */
130
125
  return fcntl(fd, F_SETFD, ret | FD_CLOEXEC);
131
126
}
132
127
 
133
 
const char *argp_program_version = "plugbasedclient 0.9";
134
 
const char *argp_program_bug_address = "<mandos@fukt.bsnet.se>";
135
 
 
136
 
process *process_list = NULL;
137
 
 
138
 
/* Mark a process as completed when it exits, and save its exit
139
 
   status. */
140
 
void handle_sigchld(__attribute__((unused)) int sig){
141
 
  process *proc = process_list;
142
 
  int status;
143
 
  pid_t pid = wait(&status);
144
 
  while(proc != NULL and proc->pid != pid){
145
 
    proc = proc->next;
146
 
  }
147
 
  if(proc == NULL){
148
 
    /* Process not found in process list */
149
 
    return;
150
 
  }
151
 
  proc->status = status;
152
 
  proc->completed = true;
153
 
}
 
128
 
 
129
#define BUFFER_SIZE 256
 
130
 
 
131
const char *argp_program_version =
 
132
  "plugbasedclient 0.9";
 
133
const char *argp_program_bug_address =
 
134
  "<mandos@fukt.bsnet.se>";
154
135
 
155
136
int main(int argc, char *argv[]){
156
137
  const char *plugindir = "/conf/conf.d/mandos/plugins.d";
157
138
  size_t d_name_len;
158
 
  DIR *dir = NULL;
 
139
  DIR *dir;
159
140
  struct dirent *dirst;
160
141
  struct stat st;
161
142
  fd_set rfds_all;
162
143
  int ret, maxfd = 0;
163
 
  uid_t uid = 65534;
164
 
  gid_t gid = 65534;
 
144
  process *process_list = NULL;
165
145
  bool debug = false;
166
146
  int exitstatus = EXIT_SUCCESS;
167
 
  struct sigaction old_sigchld_action;
168
 
  struct sigaction sigchld_action = { .sa_handler = handle_sigchld,
169
 
                                      .sa_flags = SA_NOCLDSTOP };
170
 
  char *plus_options = NULL;
171
 
  char **plus_argv = NULL;
172
 
  
173
 
  /* Establish a signal handler */
174
 
  sigemptyset(&sigchld_action.sa_mask);
175
 
  ret = sigaddset(&sigchld_action.sa_mask, SIGCHLD);
176
 
  if(ret < 0){
177
 
    perror("sigaddset");
178
 
    exit(EXIT_FAILURE);
179
 
  }
180
 
  ret = sigaction(SIGCHLD, &sigchld_action, &old_sigchld_action);
181
 
  if(ret < 0){
182
 
    perror("sigaction");
183
 
    exit(EXIT_FAILURE);
184
 
  }
185
147
  
186
148
  /* The options we understand. */
187
149
  struct argp_option options[] = {
197
159
    { .name = "plugin-dir", .key = 128,
198
160
      .arg = "DIRECTORY",
199
161
      .doc = "Specify a different plugin directory", .group = 2 },
200
 
    { .name = "userid", .key = 129,
201
 
      .arg = "ID", .flags = 0,
202
 
      .doc = "User ID the plugins will run as", .group = 2 },
203
 
    { .name = "groupid", .key = 130,
204
 
      .arg = "ID", .flags = 0,
205
 
      .doc = "Group ID the plugins will run as", .group = 2 },
206
 
    { .name = "debug", .key = 131,
 
162
    { .name = "debug", .key = 129,
207
163
      .doc = "Debug mode", .group = 3 },
208
164
    { .name = NULL }
209
165
  };
210
166
  
211
167
  error_t parse_opt (int key, char *arg, struct argp_state *state) {
212
 
    /* Get the INPUT argument from `argp_parse', which we know is a
213
 
       pointer to our plugin list pointer. */
 
168
       /* Get the INPUT argument from `argp_parse', which we
 
169
          know is a pointer to our plugin list pointer. */
214
170
    plugin **plugins = state->input;
215
171
    switch (key) {
216
172
    case 'g':
219
175
        do{
220
176
          addargument(getplugin(NULL, plugins), p);
221
177
          p = strtok(NULL, ",");
222
 
        } while (p != NULL);
 
178
        } while (p);
223
179
      }
224
180
      break;
225
181
    case 'o':
226
182
      if (arg != NULL){
227
183
        char *name = strtok(arg, ":");
228
184
        char *p = strtok(NULL, ":");
229
 
        if(p != NULL){
 
185
        if(p){
230
186
          p = strtok(p, ",");
231
187
          do{
232
188
            addargument(getplugin(name, plugins), p);
233
189
            p = strtok(NULL, ",");
234
 
          } while (p != NULL);
 
190
          } while (p);
235
191
        }
236
192
      }
237
193
      break;
244
200
      plugindir = arg;
245
201
      break;
246
202
    case 129:
247
 
      uid = (uid_t)strtol(arg, NULL, 10);
248
 
      break;
249
 
    case 130:
250
 
      gid = (gid_t)strtol(arg, NULL, 10);
251
 
      break;
252
 
    case 131:
253
203
      debug = true;
254
204
      break;
255
205
    case ARGP_KEY_ARG:
256
 
      if(plus_options != NULL or arg == NULL or arg[0] != '+'){
257
 
        argp_usage (state);
258
 
      }
259
 
      plus_options = arg;
 
206
      argp_usage (state);
260
207
      break;
261
208
    case ARGP_KEY_END:
262
209
      break;
269
216
  plugin *plugin_list = NULL;
270
217
  
271
218
  struct argp argp = { .options = options, .parser = parse_opt,
272
 
                       .args_doc = "[+PLUS_SEPARATED_OPTIONS]",
 
219
                       .args_doc = "",
273
220
                       .doc = "Mandos plugin runner -- Run plugins" };
274
221
  
275
 
  argp_parse (&argp, argc, argv, 0, 0, &plugin_list);  
276
 
  
277
 
  if(plus_options){
278
 
    /* This is a mangled argument in the form of
279
 
     "+--option+--other-option=parameter+--yet-another-option", etc */
280
 
    /* Make new argc and argv vars, and call argp_parse() again. */
281
 
    plus_options++;             /* skip the first '+' character */
282
 
    const char delims[] = "+";
283
 
    char *arg;
284
 
    int new_argc = 1;
285
 
    plus_argv = malloc(sizeof(char*) * 2);
286
 
    if(plus_argv == NULL){
287
 
      perror("malloc");
288
 
      exitstatus = EXIT_FAILURE;
289
 
      goto end;
290
 
    }
291
 
    plus_argv[0] = argv[0];
292
 
    plus_argv[1] = NULL;
293
 
    arg = strtok(plus_options, delims); /* Get first argument */
294
 
    while(arg != NULL){
295
 
      new_argc++;
296
 
      plus_argv = realloc(plus_argv, sizeof(char *)
297
 
                         * ((unsigned int) new_argc + 1));
298
 
      if(plus_argv == NULL){
299
 
        perror("malloc");
300
 
        exitstatus = EXIT_FAILURE;
301
 
        goto end;
302
 
      }
303
 
      plus_argv[new_argc-1] = arg;
304
 
      plus_argv[new_argc] = NULL;
305
 
      arg = strtok(NULL, delims); /* Get next argument */
306
 
    }
307
 
    argp_parse (&argp, new_argc, plus_argv, 0, 0, &plugin_list);  
308
 
  }
 
222
  argp_parse (&argp, argc, argv, 0, 0, &plugin_list);
309
223
  
310
224
  if(debug){
311
225
    for(plugin *p = plugin_list; p != NULL; p=p->next){
317
231
    }
318
232
  }
319
233
  
320
 
  ret = setuid(uid);
321
 
  if (ret == -1){
322
 
    perror("setuid");
323
 
  }
324
 
  
325
 
  setgid(gid);
326
 
  if (ret == -1){
327
 
    perror("setgid");
328
 
  }
329
 
  
330
234
  dir = opendir(plugindir);
331
235
  if(dir == NULL){
332
236
    perror("Could not open plugin dir");
333
237
    exitstatus = EXIT_FAILURE;
334
238
    goto end;
335
239
  }
336
 
  
337
240
  /* Set the FD_CLOEXEC flag on the directory, if possible */
338
241
  {
339
242
    int dir_fd = dirfd(dir);
347
250
    }
348
251
  }
349
252
  
 
253
  if(dir == NULL){
 
254
    fprintf(stderr, "Can not open directory\n");
 
255
    return EXIT_FAILURE;
 
256
  }
 
257
  
350
258
  FD_ZERO(&rfds_all);
351
259
  
352
260
  while(true){
410
318
      exitstatus = EXIT_FAILURE;
411
319
      goto end;
412
320
    }
413
 
    strcpy(filename, plugindir); /* Spurious warning */
414
 
    strcat(filename, "/");      /* Spurious warning */
415
 
    strcat(filename, dirst->d_name); /* Spurious warning */
416
 
    
 
321
    strcpy(filename, plugindir);
 
322
    strcat(filename, "/");
 
323
    strcat(filename, dirst->d_name);    
 
324
 
417
325
    stat(filename, &st);
418
 
    
 
326
 
419
327
    if (not S_ISREG(st.st_mode) or (access(filename, X_OK) != 0)){
420
328
      if(debug){
421
329
        fprintf(stderr, "Ignoring plugin dir entry \"%s\""
422
330
                " with bad type or mode\n", filename);
423
331
      }
424
 
      free(filename);
425
332
      continue;
426
333
    }
427
334
    if(getplugin(dirst->d_name, &plugin_list)->disabled){
429
336
        fprintf(stderr, "Ignoring disabled plugin \"%s\"\n",
430
337
                dirst->d_name);
431
338
      }
432
 
      free(filename);
433
339
      continue;
434
340
    }
435
341
    plugin *p = getplugin(dirst->d_name, &plugin_list);
459
365
      exitstatus = EXIT_FAILURE;
460
366
      goto end;
461
367
    }
462
 
    /* Block SIGCHLD until process is safely in process list */
463
 
    ret = sigprocmask (SIG_BLOCK, &sigchld_action.sa_mask, NULL);
464
 
    if(ret < 0){
465
 
      perror("sigprocmask");
466
 
      exitstatus = EXIT_FAILURE;
467
 
      goto end;
468
 
    }
469
368
    // Starting a new process to be watched
470
369
    pid_t pid = fork();
471
370
    if(pid == 0){
472
371
      /* this is the child process */
473
 
      ret = sigaction(SIGCHLD, &old_sigchld_action, NULL);
474
 
      if(ret < 0){
475
 
        perror("sigaction");
476
 
        _exit(EXIT_FAILURE);
477
 
      }
478
 
      ret = sigprocmask (SIG_UNBLOCK, &sigchld_action.sa_mask, NULL);
479
 
      if(ret < 0){
480
 
        perror("sigprocmask");
481
 
        _exit(EXIT_FAILURE);
482
 
      }
483
372
      dup2(pipefd[1], STDOUT_FILENO); /* replace our stdout */
484
373
      
485
374
      if(dirfd(dir) < 0){
486
375
        /* If dir has no file descriptor, we could not set FD_CLOEXEC
487
 
           above and must now close it manually here. */
 
376
           and must close it manually  */
488
377
        closedir(dir);
489
378
      }
490
379
      if(execv(filename, p->argv) < 0){
494
383
      /* no return */
495
384
    }
496
385
    /* parent process */
497
 
    free(filename);
498
386
    close(pipefd[1]);           /* close unused write end of pipe */
499
387
    process *new_process = malloc(sizeof(process));
500
388
    if (new_process == NULL){
501
389
      perror("malloc");
502
 
      ret = sigprocmask (SIG_UNBLOCK, &sigchld_action.sa_mask, NULL);
503
 
      if(ret < 0){
504
 
        perror("sigprocmask");
505
 
      }
506
390
      exitstatus = EXIT_FAILURE;
507
391
      goto end;
508
392
    }
510
394
    *new_process = (struct process){ .pid = pid,
511
395
                                     .fd = pipefd[0],
512
396
                                     .next = process_list };
513
 
    // List handling
514
 
    process_list = new_process;
515
 
    /* Unblock SIGCHLD so signal handler can be run if this process
516
 
       has already completed */
517
 
    ret = sigprocmask (SIG_UNBLOCK, &sigchld_action.sa_mask, NULL);
518
 
    if(ret < 0){
519
 
      perror("sigprocmask");
520
 
      exitstatus = EXIT_FAILURE;
521
 
      goto end;
522
 
    }
523
 
    
524
397
    FD_SET(new_process->fd, &rfds_all);
525
398
    
526
399
    if (maxfd < new_process->fd){
527
400
      maxfd = new_process->fd;
528
401
    }
529
402
    
 
403
    //List handling
 
404
    process_list = new_process;
530
405
  }
531
406
  
532
407
  /* Free the plugin list */
537
412
  }
538
413
  
539
414
  closedir(dir);
540
 
  dir = NULL;
541
415
  
542
416
  if (process_list == NULL){
543
417
    fprintf(stderr, "No plugin processes started, exiting\n");
544
 
    exitstatus = EXIT_FAILURE;
545
 
    goto end;
 
418
    return EXIT_FAILURE;
546
419
  }
547
 
  while(process_list){
 
420
  while(true){
548
421
    fd_set rfds = rfds_all;
549
422
    int select_ret = select(maxfd+1, &rfds, NULL, NULL, NULL);
550
423
    if (select_ret == -1){
552
425
      exitstatus = EXIT_FAILURE;
553
426
      goto end;
554
427
    }
555
 
    /* OK, now either a process completed, or something can be read
556
 
       from one of them */
557
428
    for(process *proc = process_list; proc ; proc = proc->next){
558
 
      /* Is this process completely done? */
559
 
      if(proc->eof and proc->completed){
560
 
        /* Only accept the plugin output if it exited cleanly */
561
 
        if(not WIFEXITED(proc->status)
562
 
           or WEXITSTATUS(proc->status) != 0){
563
 
          /* Bad exit by plugin */
564
 
          if(debug){
565
 
            if(WIFEXITED(proc->status)){
566
 
              fprintf(stderr, "Plugin %d exited with status %d\n",
567
 
                      proc->pid, WEXITSTATUS(proc->status));
568
 
            } else if(WIFSIGNALED(proc->status)) {
569
 
              fprintf(stderr, "Plugin %d killed by signal %d\n",
570
 
                      proc->pid, WTERMSIG(proc->status));
571
 
            } else if(WCOREDUMP(proc->status)){
572
 
              fprintf(stderr, "Plugin %d dumped core\n", proc->pid);
573
 
            }
574
 
          }
575
 
          /* Remove the plugin */
576
 
          FD_CLR(proc->fd, &rfds_all);
577
 
          /* Block signal while modifying process_list */
578
 
          ret = sigprocmask (SIG_BLOCK, &sigchld_action.sa_mask, NULL);
579
 
          if(ret < 0){
580
 
            perror("sigprocmask");
581
 
            exitstatus = EXIT_FAILURE;
582
 
            goto end;
583
 
          }
584
 
          /* Delete this process entry from the list */
585
 
          if(process_list == proc){
586
 
            /* First one - simple */
587
 
            process_list = proc->next;
588
 
          } else {
589
 
            /* Second one or later */
590
 
            for(process *p = process_list; p != NULL; p = p->next){
591
 
              if(p->next == proc){
592
 
                p->next = proc->next;
593
 
                break;
594
 
              }
595
 
            }
596
 
          }
597
 
          /* We are done modifying process list, so unblock signal */
598
 
          ret = sigprocmask (SIG_UNBLOCK, &sigchld_action.sa_mask,
599
 
                             NULL);
600
 
          if(ret < 0){
601
 
            perror("sigprocmask");
602
 
          }
603
 
          free(proc->buffer);
604
 
          free(proc);
605
 
          /* We deleted this process from the list, so we can't go
606
 
             proc->next.  Therefore, start over from the beginning of
607
 
             the process list */
608
 
          break;
609
 
        }
610
 
        /* This process exited nicely, so print its buffer */
611
 
        for(size_t written = 0; written < proc->buffer_length;
612
 
            written += (size_t)ret){
613
 
          ret = TEMP_FAILURE_RETRY(write(STDOUT_FILENO,
614
 
                                         proc->buffer + written,
615
 
                                         proc->buffer_length
616
 
                                         - written));
617
 
          if(ret < 0){
618
 
            perror("write");
619
 
            exitstatus = EXIT_FAILURE;
620
 
            goto end;
621
 
          }
622
 
        }
623
 
        goto end;
624
 
      }
625
 
      /* This process has not completed.  Does it have any output? */
626
 
      if(proc->eof or not FD_ISSET(proc->fd, &rfds)){
627
 
        /* This process had nothing to say at this time */
 
429
      if(not FD_ISSET(proc->fd, &rfds)){
628
430
        continue;
629
431
      }
630
 
      /* Before reading, make the process' data buffer large enough */
631
432
      if(proc->buffer_length + BUFFER_SIZE > proc->buffer_size){
632
433
        proc->buffer = realloc(proc->buffer, proc->buffer_size
633
434
                               + (size_t) BUFFER_SIZE);
638
439
        }
639
440
        proc->buffer_size += BUFFER_SIZE;
640
441
      }
641
 
      /* Read from the process */
642
442
      ret = read(proc->fd, proc->buffer + proc->buffer_length,
643
443
                 BUFFER_SIZE);
644
444
      if(ret < 0){
645
 
        /* Read error from this process; ignore the error */
 
445
        /* Read error from this process; ignore it */
646
446
        continue;
647
447
      }
 
448
      proc->buffer_length += (size_t) ret;
648
449
      if(ret == 0){
649
450
        /* got EOF */
650
 
        proc->eof = true;
651
 
      } else {
652
 
        proc->buffer_length += (size_t) ret;
 
451
        /* wait for process exit */
 
452
        int status;
 
453
        waitpid(proc->pid, &status, 0);
 
454
        if(not WIFEXITED(status) or WEXITSTATUS(status) != 0){
 
455
          FD_CLR(proc->fd, &rfds_all);
 
456
          continue;
 
457
        }
 
458
        for(size_t written = 0;
 
459
            written < proc->buffer_length; written += (size_t)ret){
 
460
          ret = TEMP_FAILURE_RETRY(write(STDOUT_FILENO,
 
461
                                         proc->buffer + written,
 
462
                                         proc->buffer_length
 
463
                                         - written));
 
464
          if(ret < 0){
 
465
            perror("write");
 
466
            exitstatus = EXIT_FAILURE;
 
467
            goto end;
 
468
          }
 
469
        }
 
470
        goto end;
653
471
      }
654
472
    }
655
473
  }
656
 
  if(process_list == NULL){
657
 
    fprintf(stderr, "All plugin processes failed, exiting\n");
658
 
    exitstatus = EXIT_FAILURE;
659
 
  }
660
474
  
661
475
 end:
662
 
  /* Restore old signal handler */
663
 
  sigaction(SIGCHLD, &old_sigchld_action, NULL);
664
 
  
665
 
  free(plus_argv);
666
 
  
667
 
  /* Free the plugin list */
668
 
  for(plugin *next; plugin_list != NULL; plugin_list = next){
669
 
    next = plugin_list->next;
670
 
    free(plugin_list->argv);
671
 
    free(plugin_list);
672
 
  }
673
 
  
674
 
  if(dir != NULL){
675
 
    closedir(dir);
676
 
  }
677
 
  
678
 
  /* Free the process list and kill the processes */
679
476
  for(process *next; process_list != NULL; process_list = next){
680
 
    next = process_list->next;
681
477
    close(process_list->fd);
682
478
    kill(process_list->pid, SIGTERM);
683
479
    free(process_list->buffer);
684
480
    free(process_list);
685
481
  }
686
482
  
687
 
  /* Wait for any remaining child processes to terminate */
688
 
  do{
689
 
    ret = wait(NULL);
690
 
  } while(ret >= 0);
691
 
  if(errno != ECHILD){
692
 
    perror("wait");
693
 
  }
694
 
  
 
483
  while(true){
 
484
    int status;
 
485
    ret = wait(&status);
 
486
    if (ret == -1){
 
487
      if(errno != ECHILD){
 
488
        perror("wait");
 
489
      }
 
490
      break;
 
491
    }
 
492
  }  
695
493
  return exitstatus;
696
494
}