diff options
author | Christian Göttsche <cgzones@googlemail.com> | 2023-02-08 18:07:59 +0100 |
---|---|---|
committer | Christian Göttsche <cgzones@googlemail.com> | 2023-02-28 15:11:18 +0100 |
commit | 81e74d598b080de5c6a66585ebf2a9d6d1186f00 (patch) | |
tree | 8430c04cbba25cc8a6fce14446152c5116ed0325 | |
parent | 8a3f0810397c992a1155d815b1dd40b0b8fed59d (diff) | |
download | pam-81e74d598b080de5c6a66585ebf2a9d6d1186f00.tar.gz pam-81e74d598b080de5c6a66585ebf2a9d6d1186f00.tar.bz2 pam-81e74d598b080de5c6a66585ebf2a9d6d1186f00.zip |
pam_env: override undefined pointer after asprintf failure
On failure the content of the string pointer passed to asprintf(3) is
undefined. Set to NULL before free'ing the parent array.
-rw-r--r-- | modules/pam_env/pam_env.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/modules/pam_env/pam_env.c b/modules/pam_env/pam_env.c index a25e195b..2868cfce 100644 --- a/modules/pam_env/pam_env.c +++ b/modules/pam_env/pam_env.c @@ -265,6 +265,7 @@ econf_read_file(const pam_handle_t *pamh, const char *filename, const char *deli pam_syslog(pamh, LOG_ERR, "Cannot allocate memory."); econf_free(keys); econf_freeFile(key_file); + (*lines)[i] = NULL; free_string_array(*lines); free (val); return PAM_BUF_ERR; |