diff options
| author | Tomas Mraz <tmraz@fedoraproject.org> | 2011-06-02 21:55:41 +0200 |
|---|---|---|
| committer | Tomas Mraz <tmraz@fedoraproject.org> | 2011-06-02 21:55:41 +0200 |
| commit | 26747b9b490d190dd20543ea9cbde082ae667402 (patch) | |
| tree | 685f9db583b3428a0d098e2c5acacfb3847f7e14 | |
| parent | 48590abce86b34e55c84f71424449f16d285eaf2 (diff) | |
| download | pam-26747b9b490d190dd20543ea9cbde082ae667402.tar.gz pam-26747b9b490d190dd20543ea9cbde082ae667402.tar.bz2 pam-26747b9b490d190dd20543ea9cbde082ae667402.zip | |
Guard for pam_get_user() error in pam_filter module.
| -rw-r--r-- | ChangeLog | 3 | ||||
| -rw-r--r-- | modules/pam_filter/pam_filter.c | 4 |
2 files changed, 5 insertions, 2 deletions
@@ -13,6 +13,9 @@ on NULL strings. (password_check): Guard for NULLs returned from memory allocation. + * modules/pam_filter/pam_filter.c (process_args): Guard for error return + from pam_get_user(). + 2011-05-30 Thorsten Kukuk <kukuk@thkukuk.de> * modules/pam_timestamp/pam_timestamp.c (main): Remove unsused diff --git a/modules/pam_filter/pam_filter.c b/modules/pam_filter/pam_filter.c index 2f290fd5..da98148f 100644 --- a/modules/pam_filter/pam_filter.c +++ b/modules/pam_filter/pam_filter.c @@ -177,8 +177,8 @@ static int process_args(pam_handle_t *pamh #define USER_OFFSET 5 /* strlen('USER='); */ #define USER_NAME "USER=" - pam_get_user(pamh, &user, NULL); - if (user == NULL) { + if (pam_get_user(pamh, &user, NULL) != PAM_SUCCESS || + user == NULL) { user = "<unknown>"; } size = USER_OFFSET+strlen(user); |