/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 plugins.d/password-prompt.c

  • Committer: Björn Påhlsson
  • Date: 2008-09-04 19:59:05 UTC
  • mto: (237.7.1 mandos) (24.1.154 mandos)
  • mto: This revision was merged to the branch mainline in revision 164.
  • Revision ID: belorn@braxen-20080904195905-a3qfv1np32auumwm
fixed two bugs:
      A overbuffer flow in enviro variables causing seg fault
      A incorrect use of strsep causing seg fault

Show diffs side-by-side

added added

removed removed

Lines of Context:
plugins.d/password-prompt.c
2
2
/*
3
3
 * Passprompt - Read a password from the terminal and print it
4
4
 *
5
 
 * Copyright © 2008 Teddy Hogeborn & Björn Påhlsson
 
5
 * Copyright © 2007-2008 Teddy Hogeborn & Björn Påhlsson
6
6
 * 
7
7
 * This program is free software: you can redistribute it and/or
8
8
 * modify it under the terms of the GNU General Public License as
216
216
      status = EXIT_SUCCESS;
217
217
      /* Make n = data size instead of allocated buffer size */
218
218
      n = (size_t)ret;
219
 
      /* Strip final newline */
220
 
      if(n>0 and buffer[n-1] == '\n'){
221
 
        buffer[n-1] = '\0';     /* not strictly necessary */
222
 
        n--;
223
 
      }
224
219
      size_t written = 0;
225
220
      while(written < n){
226
221
        ret = write(STDOUT_FILENO, buffer + written, n - written);
249
244
      fprintf(stderr, "getline() returned 0, retrying.\n");
250
245
    }
251
246
  }
252
 
 
253
 
  free(buffer);
254
247
  
255
248
  if (debug){
256
249
    fprintf(stderr, "Restoring terminal attributes\n");
263
256
    fprintf(stderr, "%s is exiting with status %d\n", argv[0],
264
257
            status);
265
258
  }
266
 
  if(status == EXIT_SUCCESS){
267
 
    fputc('\n', stderr);
268
 
  }
269
259
  
270
260
  return status;
271
261
}