summaryrefslogtreecommitdiff
path: root/modules/pam_timestamp
diff options
context:
space:
mode:
authorTomas Mraz <tmraz@fedoraproject.org>2011-06-15 20:58:32 +0200
committerTomas Mraz <tmraz@fedoraproject.org>2011-06-15 20:58:32 +0200
commit2daaafacf771746a11849ada2166f8ac2bab1348 (patch)
tree961f3161a4e52b047306ad121c4b669c5ef14e62 /modules/pam_timestamp
parent53d6722ab192193c77d187645a31949d3de65f2b (diff)
Avoid memleaks and fd leak in error paths.
Diffstat (limited to 'modules/pam_timestamp')
-rw-r--r--modules/pam_timestamp/pam_timestamp.c3
1 files changed, 3 insertions, 0 deletions
diff --git a/modules/pam_timestamp/pam_timestamp.c b/modules/pam_timestamp/pam_timestamp.c
index 7bcf3d12..51937333 100644
--- a/modules/pam_timestamp/pam_timestamp.c
+++ b/modules/pam_timestamp/pam_timestamp.c
@@ -483,6 +483,7 @@ pam_sm_authenticate(pam_handle_t *pamh, int flags, int argc, const char **argv)
pam_syslog(pamh, LOG_NOTICE, "timestamp file `%s' is "
"corrupted", path);
close(fd);
+ free(mac);
free(message);
return PAM_AUTH_ERR;
}
@@ -636,6 +637,8 @@ pam_sm_open_session(pam_handle_t *pamh, int flags UNUSED, int argc, const char *
"error setting ownership of `%s': %m",
path);
}
+ close(fd);
+ free(text);
return PAM_SESSION_ERR;
}