From f2b7f432bc20a90b836c6c2d2dba53979296ccc0 Mon Sep 17 00:00:00 2001 From: "Dmitry V. Levin" Date: Mon, 18 Feb 2008 13:18:43 +0000 Subject: Relevant BUGIDs: Purpose of commit: bugfix Commit summary: --------------- 2008-02-18 Dmitry V. Levin * modules/pam_exec/pam_exec.c (call_exec): Fix asprintf return code check. --- modules/pam_exec/pam_exec.c | 9 ++++----- 1 file changed, 4 insertions(+), 5 deletions(-) (limited to 'modules/pam_exec') diff --git a/modules/pam_exec/pam_exec.c b/modules/pam_exec/pam_exec.c index 766c0a06..14dddd54 100644 --- a/modules/pam_exec/pam_exec.c +++ b/modules/pam_exec/pam_exec.c @@ -244,8 +244,8 @@ call_exec (pam_handle_t *pamh, int argc, const char **argv) if (tmp == NULL) { free(envlist); - pam_syslog (pamh, LOG_ERR, "realloc environment failed : %m"); - exit (ENOMEM); + pam_syslog (pamh, LOG_ERR, "realloc environment failed: %m"); + exit (ENOMEM); } envlist = tmp; for (i = 0; i < nitems; ++i) @@ -255,11 +255,10 @@ call_exec (pam_handle_t *pamh, int argc, const char **argv) if (pam_get_item(pamh, env_items[i].item, &item) != PAM_SUCCESS || item == NULL) continue; - asprintf(&envstr, "%s=%s", env_items[i].name, (const char *)item); - if (envstr == NULL) + if (asprintf(&envstr, "%s=%s", env_items[i].name, (const char *)item) < 0) { free(envlist); - pam_syslog (pamh, LOG_ERR, "prepare environment failed : %m"); + pam_syslog (pamh, LOG_ERR, "prepare environment failed: %m"); exit (ENOMEM); } envlist[envlen++] = envstr; -- cgit v1.2.3