diff options
author | dtucker <dtucker> | 2006-05-17 12:24:56 +0000 |
---|---|---|
committer | dtucker <dtucker> | 2006-05-17 12:24:56 +0000 |
commit | d5c6670e55ab9552130bbb77408a5531fa917da9 (patch) | |
tree | df19085dad561d7e064caaeb86f73d6d633f1f37 /ssh-rand-helper.c | |
parent | 89ae52f23852670d2e97547e7c6b5647163b0263 (diff) | |
download | openssh-d5c6670e55ab9552130bbb77408a5531fa917da9.tar.gz |
- (dtucker) [ssh-rand-helper.c] Check return code of mkdir and fix file
pointer leak. From kjhall at us.ibm.com, found by coverity.
Diffstat (limited to 'ssh-rand-helper.c')
-rw-r--r-- | ssh-rand-helper.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/ssh-rand-helper.c b/ssh-rand-helper.c index 3a4a165f..d88a246e 100644 --- a/ssh-rand-helper.c +++ b/ssh-rand-helper.c @@ -573,7 +573,8 @@ prng_write_seedfile(void) /* Try to ensure that the parent directory is there */ snprintf(filename, sizeof(filename), "%.512s/%s", pw->pw_dir, _PATH_SSH_USER_DIR); - mkdir(filename, 0700); + if (mkdir(filename, 0700) < 0) + fatal("mkdir: %s", strerror(errno)); snprintf(filename, sizeof(filename), "%.512s/%s", pw->pw_dir, SSH_PRNG_SEED_FILE); @@ -782,6 +783,7 @@ prng_read_commands(char *cmdfilename) debug("Loaded %d entropy commands from %.100s", cur_cmd, cmdfilename); + fclose(f); return cur_cmd < MIN_ENTROPY_SOURCES ? -1 : 0; } |