/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: 2008-09-06 15:34:21 UTC
  • Revision ID: teddy@fukt.bsnet.se-20080906153421-605ydhwyo404krh9
* initramfs-tools-hook: Use long options where available.  Use only
                        one "test" invocation instead of several.

* mandos-keygen: Use only one "test" invocation instead of several.
                 Use long options where available.  Strip newline from
                 entered password.

* plugin-runner.c (print_out_password): Do not strip trailing newline.
  (fallback): Strip trailing newline from getpass().

* plugin-runner.xml (PLUGINS): Plugins should not print trailing
                               newlines.

* plugins.d/password-prompt.c (main): Strip trailing newline.

* plugins.d/password-request.c (main): Bug fix: Read "pubkey.txt", not
                                       "pupkey.txt".

Show diffs side-by-side

added added

removed removed

Lines of Context:
plugin-runner.c
96
96
 
97
97
static plugin *plugin_list = NULL;
98
98
 
99
 
/* Gets a existing plugin based on name,
 
99
/* Gets an existing plugin based on name,
100
100
   or if none is found, creates a new one */
101
101
static plugin *getplugin(char *name){
102
102
  /* Check for exiting plugin with that name */
118
118
      return NULL;
119
119
    }
120
120
  }
121
 
 
 
121
  
122
122
  *new_plugin = (plugin) { .name = copy_name,
123
123
                           .argc = 1,
124
124
                           .disabled = false,
187
187
  size_t namelen = (size_t)(strchrnul(def, '=') - def);
188
188
  /* Search for this environment variable */
189
189
  for(char **e = p->environ; *e != NULL; e++){
190
 
    if(strncmp(*e, def, namelen+1) == 0){
 
190
    if(strncmp(*e, def, namelen + 1) == 0){
191
191
      /* It already exists */
192
192
      if(replace){
193
 
        char *new = realloc(*e, strlen(def));
 
193
        char *new = realloc(*e, strlen(def) + 1);
194
194
        if(new == NULL){
195
195
          return false;
196
196
        }
238
238
      /* No child processes */
239
239
      break;
240
240
    }
241
 
 
 
241
    
242
242
    /* A child exited, find it in process_list */
243
243
    while(proc != NULL and proc->pid != pid){
244
244
      proc = proc->next;
255
255
/* Prints out a password to stdout */
256
256
bool print_out_password(const char *buffer, size_t length){
257
257
  ssize_t ret;
258
 
  if(length>0 and buffer[length-1] == '\n'){
259
 
    length--;
260
 
  }
261
258
  for(size_t written = 0; written < length; written += (size_t)ret){
262
259
    ret = TEMP_FAILURE_RETRY(write(STDOUT_FILENO, buffer + written,
263
260
                                   length - written));
410
407
    case 'o':                   /* --options-for */
411
408
      if (arg != NULL){
412
409
        char *p_name = strsep(&arg, ":");
413
 
        if(p_name[0] == '\0'){
 
410
        if(p_name[0] == '\0' or arg == NULL){
414
411
          break;
415
412
        }
416
413
        char *opt = strsep(&arg, ":");
417
 
        if(opt[0] == '\0'){
 
414
        if(opt[0] == '\0' or opt == NULL){
418
415
          break;
419
416
        }
420
 
        if(opt != NULL){
421
 
          char *p;
422
 
          while((p = strsep(&opt, ",")) != NULL){
423
 
            if(p[0] == '\0'){
424
 
              continue;
425
 
            }
426
 
            if(not add_argument(getplugin(p_name), p)){
427
 
              perror("add_argument");
428
 
              return ARGP_ERR_UNKNOWN;
429
 
            }
 
417
        char *p;
 
418
        while((p = strsep(&opt, ",")) != NULL){
 
419
          if(p[0] == '\0'){
 
420
            continue;
 
421
          }
 
422
          if(not add_argument(getplugin(p_name), p)){
 
423
            perror("add_argument");
 
424
            return ARGP_ERR_UNKNOWN;
430
425
          }
431
426
        }
432
427
      }
1043
1038
    bool bret;
1044
1039
    fprintf(stderr, "Going to fallback mode using getpass(3)\n");
1045
1040
    char *passwordbuffer = getpass("Password: ");
1046
 
    bret = print_out_password(passwordbuffer, strlen(passwordbuffer));
 
1041
    size_t len = strlen(passwordbuffer);
 
1042
    /* Strip trailing newline */
 
1043
    if(len > 0 and passwordbuffer[len-1] == '\n'){
 
1044
      passwordbuffer[len-1] = '\0'; /* not strictly necessary */
 
1045
      len--;
 
1046
    }
 
1047
    bret = print_out_password(passwordbuffer, len);
1047
1048
    if(not bret){
1048
1049
      perror("print_out_password");
1049
1050
      exitstatus = EXIT_FAILURE;