diff options
| author | Alexander Bokovoy <abokovoy@redhat.com> | 2013-09-02 16:39:30 +0300 |
|---|---|---|
| committer | Alexander Bokovoy <abokovoy@redhat.com> | 2013-09-02 16:39:30 +0300 |
| commit | b58a7192b683fe7ab4c8c3d95d73ed4223f7247a (patch) | |
| tree | 69df5a9a282a9ee23be62576ecd21a92b1520f6d | |
| parent | ec00422ceeabc1296031675ff0cbd559cbd23806 (diff) | |
| download | slapi-nis-b58a7192b683fe7ab4c8c3d95d73ed4223f7247a.tar.gz slapi-nis-b58a7192b683fe7ab4c8c3d95d73ed4223f7247a.tar.xz slapi-nis-b58a7192b683fe7ab4c8c3d95d73ed4223f7247a.zip | |
Coverity#11940: do not leak memory in the pam wrapper testcoverity
| -rw-r--r-- | tests/wrap-pam.c | 8 |
1 files 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); |