diff options
| author | Thorsten Kukuk <kukuk@thkukuk.de> | 2010-09-30 13:47:14 +0000 |
|---|---|---|
| committer | Thorsten Kukuk <kukuk@thkukuk.de> | 2010-09-30 13:47:14 +0000 |
| commit | e39126075bf9c8434a9ad4197a4869444afa23cb (patch) | |
| tree | 5bdf2882c0aae635535f28e2860e89f653142f17 | |
| parent | 4d5576d109d315482038ebdfb92b050df7b5761c (diff) | |
Relevant BUGIDs:
Purpose of commit: cleanup
Commit summary:
---------------
2010-09-30 Thorsten Kukuk <kukuk@thkukuk.de>
* modules/pam_xauth/pam_xauth.c (pam_sm_close_session): Warn if
unlink() fails.
| -rw-r--r-- | ChangeLog | 5 | ||||
| -rw-r--r-- | modules/pam_xauth/pam_xauth.c | 3 |
2 files changed, 7 insertions, 1 deletions
@@ -1,3 +1,8 @@ +2010-09-30 Thorsten Kukuk <kukuk@thkukuk.de> + + * modules/pam_xauth/pam_xauth.c (pam_sm_close_session): Warn if + unlink() fails. + 2010-09-27 Dmitry V. Levin <ldv@altlinux.org> * modules/pam_xauth/pam_xauth.c (pam_sm_close_session): Return diff --git a/modules/pam_xauth/pam_xauth.c b/modules/pam_xauth/pam_xauth.c index aab1a579..be2a2c7c 100644 --- a/modules/pam_xauth/pam_xauth.c +++ b/modules/pam_xauth/pam_xauth.c @@ -775,7 +775,8 @@ pam_sm_close_session (pam_handle_t *pamh, int flags UNUSED, if (debug) pam_syslog(pamh, LOG_DEBUG, "removing `%s'", cookiefile); fsuid = setfsuid(tpwd->pw_uid); - unlink(cookiefile); + if (unlink(cookiefile) == -1 && errno != ENOENT) + pam_syslog(pamh, LOG_WARNING, "Couldn't remove `%s': %m", cookiefile); setfsuid(fsuid); return PAM_SUCCESS; |