From 9facab2134a9e1142ab3c614e72eb25aaafd0dec Mon Sep 17 00:00:00 2001 From: Christian Göttsche Date: Sat, 5 Aug 2023 17:34:42 +0200 Subject: tests: free handles via pam_end() Destroy the pam handles via pam_end() to release all associated resources. This allows to run the test-suite with sanitizers and validates the resource cleanup in pam_end() and callees. --- tests/tst-pam_fail_delay.c | 2 ++ tests/tst-pam_getenvlist.c | 2 ++ tests/tst-pam_start.c | 4 ++++ tests/tst-pam_start_confdir.c | 6 ++++++ 4 files changed, 14 insertions(+) (limited to 'tests') diff --git a/tests/tst-pam_fail_delay.c b/tests/tst-pam_fail_delay.c index d81c5765..f166ec0d 100644 --- a/tests/tst-pam_fail_delay.c +++ b/tests/tst-pam_fail_delay.c @@ -67,6 +67,8 @@ main (void) return 1; } + pam_end (pamh, retval); + /* 2: use NULL pam handle */ retval = pam_fail_delay (NULL, 60); if (retval == PAM_SUCCESS) diff --git a/tests/tst-pam_getenvlist.c b/tests/tst-pam_getenvlist.c index a1184f1a..62aea061 100644 --- a/tests/tst-pam_getenvlist.c +++ b/tests/tst-pam_getenvlist.c @@ -130,5 +130,7 @@ main (void) free (ptr); } + pam_end (pamh, retval); + return 0; } diff --git a/tests/tst-pam_start.c b/tests/tst-pam_start.c index 8fa18f74..23af4fe8 100644 --- a/tests/tst-pam_start.c +++ b/tests/tst-pam_start.c @@ -66,6 +66,8 @@ main (void) return 1; } + pam_end (pamh, retval); + /* 2: check with NULL for service */ retval = pam_start (NULL, user, &conv, &pamh); if (retval == PAM_SUCCESS) @@ -84,6 +86,8 @@ main (void) return 1; } + pam_end (pamh, retval); + /* 4: check with NULL for conv */ retval = pam_start (service, user, NULL, &pamh); diff --git a/tests/tst-pam_start_confdir.c b/tests/tst-pam_start_confdir.c index f731b2a5..b57ea573 100644 --- a/tests/tst-pam_start_confdir.c +++ b/tests/tst-pam_start_confdir.c @@ -77,6 +77,8 @@ main (void) return 1; } + pam_end (pamh, retval); + /* 2: check with invalid service */ retval = pam_start_confdir (xservice, user, &conv, confdir, &pamh); if (retval == PAM_SUCCESS) @@ -86,6 +88,8 @@ main (void) return 1; } + pam_end (pamh, retval); + /* 3: check with invalid confdir */ retval = pam_start_confdir (service, user, &conv, xconfdir, &pamh); if (retval == PAM_SUCCESS) @@ -95,5 +99,7 @@ main (void) return 1; } + pam_end (pamh, retval); + return 0; } -- cgit v1.2.3