Before pam_end(), the return value of the previous

pam API was already checked. No need to validate it again.
2007-12-28 21:29:06 +00:00
parent 8dc4ca297c
commit dc1dccd9e2
2 changed files with 6 additions and 2 deletions
--- a/src/chpasswd.c
+++ b/src/chpasswd.c
@ -411,8 +411,7 @@ int main (int argc, char **argv)
 	pw_unlock ();

 #ifdef USE_PAM
-	if (retval == PAM_SUCCESS)
-		pam_end (pamh, PAM_SUCCESS);
+	pam_end (pamh, PAM_SUCCESS);
 #endif				/* USE_PAM */

 	return (0);