Changeset 56 in openpam

Timestamp:

Feb 13, 2002, 2:16:03 AM (19 years ago)

Author:

Dag-Erling Smørgrav

Message:

Add a missing 'continue' in the code for the PAM_SUCCESS case.
Add more debugging log messages.

Sponsored by: DARPA, NAI Labs

File:

• trunk/lib/openpam_dispatch.c (modified) (4 diffs)

trunk/lib/openpam_dispatch.c

@@ -120 +120 @@
                             primitive != PAM_SM_SETCRED)
                                 break;
+                        continue;
                 }
 
@@ -132 +133 @@
                         err = r;
                 if (chain->flag == PAM_REQUIRED && !fail) {
+                        openpam_log(PAM_LOG_DEBUG, "required module failed");
                         fail = 1;
                         err = r;
@@ -141 +143 @@
                  */
                 if (chain->flag == PAM_REQUISITE) {
+                        openpam_log(PAM_LOG_DEBUG, "requisite module failed");
                         fail = 1;
                         break;
@@ -146 +149 @@
         }
 
-        return (fail ? err : PAM_SUCCESS);
+        if (!fail)
+                err = PAM_SUCCESS;
+        openpam_log(PAM_LOG_DEBUG, "returning: %s", pam_strerror(pamh, err));
+        return (err);
 }