/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 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:
plugin-runner.c
2
2
/*
3
3
 * Mandos plugin runner - Run Mandos plugins
4
4
 *
5
 
 * Copyright © 2008-2012 Teddy Hogeborn
6
 
 * Copyright © 2008-2012 Björn Påhlsson
 
5
 * Copyright © 2008-2011 Teddy Hogeborn
 
6
 * Copyright © 2008-2011 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
426
426
    errno = 0;
427
427
    switch(key){
428
428
      char *tmp;
429
 
      intmax_t tmp_id;
 
429
      intmax_t tmpmax;
430
430
    case 'g':                   /* --global-options */
431
431
      {
432
432
        char *plugin_option;
505
505
      /* This is already done by parse_opt_config_file() */
506
506
      break;
507
507
    case 130:                   /* --userid */
508
 
      tmp_id = strtoimax(arg, &tmp, 10);
 
508
      tmpmax = strtoimax(arg, &tmp, 10);
509
509
      if(errno != 0 or tmp == arg or *tmp != '\0'
510
 
         or tmp_id != (uid_t)tmp_id){
 
510
         or tmpmax != (uid_t)tmpmax){
511
511
        argp_error(state, "Bad user ID number: \"%s\", using %"
512
512
                   PRIdMAX, arg, (intmax_t)uid);
513
513
        break;
514
514
      }
515
 
      uid = (uid_t)tmp_id;
 
515
      uid = (uid_t)tmpmax;
516
516
      break;
517
517
    case 131:                   /* --groupid */
518
 
      tmp_id = strtoimax(arg, &tmp, 10);
 
518
      tmpmax = strtoimax(arg, &tmp, 10);
519
519
      if(errno != 0 or tmp == arg or *tmp != '\0'
520
 
         or tmp_id != (gid_t)tmp_id){
 
520
         or tmpmax != (gid_t)tmpmax){
521
521
        argp_error(state, "Bad group ID number: \"%s\", using %"
522
522
                   PRIdMAX, arg, (intmax_t)gid);
523
523
        break;
524
524
      }
525
 
      gid = (gid_t)tmp_id;
 
525
      gid = (gid_t)tmpmax;
526
526
      break;
527
527
    case 132:                   /* --debug */
528
528
      debug = true;
771
771
  }
772
772
  
773
773
  /* Lower permissions */
774
 
  ret = setgid(gid);
 
774
  setgid(gid);
775
775
  if(ret == -1){
776
776
    error(0, errno, "setgid");
777
777
  }