/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

* plugin-runner.c (main): Use "sscanf" instead of "strtol"; using the
                          correct type instead of casting.
* plugins.d/mandos-cliend.c (main): Detect an empty string when
                                    parsing DH bits and port number.
* plugins.d/splashy.c (main): Use "sscanf" instead of "strtoul"; using
                              the correct type instead of casting.
* plugins.d/usplash.c (main): - '' -

Show diffs side-by-side

added added

removed removed

Lines of Context:
plugin-runner.c
309
309
  struct stat st;
310
310
  fd_set rfds_all;
311
311
  int ret, maxfd = 0;
 
312
  ssize_t sret;
312
313
  uid_t uid = 65534;
313
314
  gid_t gid = 65534;
314
315
  bool debug = false;
462
463
      /* This is already done by parse_opt_config_file() */
463
464
      break;
464
465
    case 130:                   /* --userid */
465
 
      uid = (uid_t)strtol(arg, NULL, 10);
 
466
      /* In the GNU C library, uid_t is always unsigned int */
 
467
      ret = sscanf(arg, "%ud", &uid);
 
468
      if(ret != 1){
 
469
        fprintf(stderr, "Bad user ID number: \"%s\", using %ud\n",
 
470
                arg, uid);
 
471
      }
466
472
      break;
467
473
    case 131:                   /* --groupid */
468
 
      gid = (gid_t)strtol(arg, NULL, 10);
 
474
      /* In the GNU C library, gid_t is always unsigned int */
 
475
      ret = sscanf(arg, "%ud", &gid);
 
476
      if(ret != 1){
 
477
        fprintf(stderr, "Bad group ID number: \"%s\", using %ud\n",
 
478
                arg, gid);
 
479
      }
469
480
      break;
470
481
    case 132:                   /* --debug */
471
482
      debug = true;
472
483
      break;
 
484
/*
 
485
 * When adding more options before this line, remember to also add a
 
486
 * "case" to the "parse_opt_config_file" function below.
 
487
 */
473
488
    case ARGP_KEY_ARG:
474
489
      /* Cryptsetup always passes an argument, which is an empty
475
490
         string if "none" was specified in /etc/crypttab.  So if
546
561
    char *org_line = NULL;
547
562
    char *p, *arg, *new_arg, *line;
548
563
    size_t size = 0;
549
 
    ssize_t sret;
550
564
    const char whitespace_delims[] = " \r\t\f\v\n";
551
565
    const char comment_delim[] = "#";
552
566
 
1019
1033
        proc->buffer_size += BUFFER_SIZE;
1020
1034
      }
1021
1035
      /* Read from the process */
1022
 
      ret = read(proc->fd, proc->buffer + proc->buffer_length,
1023
 
                 BUFFER_SIZE);
1024
 
      if(ret < 0){
 
1036
      sret = read(proc->fd, proc->buffer + proc->buffer_length,
 
1037
                  BUFFER_SIZE);
 
1038
      if(sret < 0){
1025
1039
        /* Read error from this process; ignore the error */
1026
1040
        proc = proc->next;
1027
1041
        continue;
1028
1042
      }
1029
 
      if(ret == 0){
 
1043
      if(sret == 0){
1030
1044
        /* got EOF */
1031
1045
        proc->eof = true;
1032
1046
      } else {
1033
 
        proc->buffer_length += (size_t) ret;
 
1047
        proc->buffer_length += (size_t) sret;
1034
1048
      }
1035
1049
    }
1036
1050
  }