From b58a7192b683fe7ab4c8c3d95d73ed4223f7247a Mon Sep 17 00:00:00 2001 From: Alexander Bokovoy Date: Mon, 2 Sep 2013 16:39:30 +0300 Subject: Coverity#11940: do not leak memory in the pam wrapper test --- tests/wrap-pam.c | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/tests/wrap-pam.c b/tests/wrap-pam.c index dc92287..45fa0fc 100644 --- a/tests/wrap-pam.c +++ b/tests/wrap-pam.c @@ -111,14 +111,16 @@ pam_start(const char *service_name, const char *user, char buf[LINE_MAX], *p, *q; pam_handle_t *ret; + if (getenv("WRAPPERS_PAM_CREDS") == NULL) { + return PAM_ABORT; + } + ret = calloc(1, sizeof(*ret)); if (ret == NULL) { return PAM_BUF_ERR; } ret->conv = *pam_conversation; - if (getenv("WRAPPERS_PAM_CREDS") == NULL) { - return PAM_ABORT; - } + fp = fopen(getenv("WRAPPERS_PAM_CREDS"), "r"); if (fp == NULL) { free(ret); -- cgit