summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorAmarnath Valluri <amarnath.valluri@intel.com>2015-08-05 15:16:51 +0200
committerTomas Mraz <tmraz@fedoraproject.org>2015-08-05 15:16:51 +0200
commitbb506282825923b89d61e96e76e2f67c67374284 (patch)
treeec26e25b152f9f0e9cb319e96d12cc3e79cf8bcd
parente89d4c97385ff8180e6e81e84c5aa745daf28a79 (diff)
pam_timestamp: Avoid leaking file descriptor.
* modules/pam_timestamp/hmacsha1.c(hmac_key_create): close 'keyfd' when failed to own it. Signed-off-by: Amarnath Valluri <amarnath.valluri@intel.com>
-rw-r--r--modules/pam_timestamp/hmacsha1.c1
1 files changed, 1 insertions, 0 deletions
diff --git a/modules/pam_timestamp/hmacsha1.c b/modules/pam_timestamp/hmacsha1.c
index 573ecf3b..3f411061 100644
--- a/modules/pam_timestamp/hmacsha1.c
+++ b/modules/pam_timestamp/hmacsha1.c
@@ -73,6 +73,7 @@ hmac_key_create(pam_handle_t *pamh, const char *filename, size_t key_size,
if (fchown(keyfd, owner, group) == -1) {
pam_syslog(pamh, LOG_ERR, "Cannot chown %s: %m", filename);
+ close(keyfd);
return;
}