commit 3475dbeb44238b5b8910cea4abfde106c6e90618 Author: Tomas Mraz tmraz@fedoraproject.org Date: Thu Jun 2 21:57:31 2011 +0200
Check for return value of pam_get_item() in pam_echo module.
ChangeLog | 3 +++ modules/pam_echo/pam_echo.c | 5 ++++- 2 files changed, 7 insertions(+), 1 deletions(-) --- diff --git a/ChangeLog b/ChangeLog index 90bb7c0..a7c485c 100644 --- a/ChangeLog +++ b/ChangeLog @@ -16,6 +16,9 @@ * modules/pam_filter/pam_filter.c (process_args): Guard for error return from pam_get_user().
+ * modules/pam_echo/pam_echo.c (replace_and_print): Guard for error return + from pam_get_item(). + 2011-05-30 Thorsten Kukuk kukuk@thkukuk.de
* modules/pam_timestamp/pam_timestamp.c (main): Remove unsused diff --git a/modules/pam_echo/pam_echo.c b/modules/pam_echo/pam_echo.c index 31ebca2..043ff70 100644 --- a/modules/pam_echo/pam_echo.c +++ b/modules/pam_echo/pam_echo.c @@ -119,7 +119,10 @@ replace_and_print (pam_handle_t *pamh, const char *mesg) str = &myhostname; } else - pam_get_item (pamh, item, &str); + { + if (pam_get_item (pamh, item, &str) != PAM_SUCCESS) + str = NULL; + } if (str == NULL) str = "(null)"; for (q = str; *q != '\0' && len < length - 1; ++q)
linux-pam-commits@lists.fedorahosted.org