Browse Source

Ticket #39 - Account Policy Plugin does not work for simple binds when PAM Pass Through Auth plugin is enabled

Bug Description:  The PAM plugin, if the bind was successful, aborts the pre and post op plugins.
                  This causes problems with post op plugins like Account Policy.

Fix Description:  In do_bind(), still call the post op plugins even if the pre-op ones fail.

https://fedorahosted.org/389/ticket/39
Mark Reynolds 14 years ago
parent
commit
a55c91ec4c
1 changed files with 3 additions and 0 deletions
  1. 3 0
      ldap/servers/slapd/bind.c

+ 3 - 0
ldap/servers/slapd/bind.c

@@ -796,6 +796,9 @@ do_bind( Slapi_PBlock *pb )
 
             slapi_pblock_set( pb, SLAPI_PLUGIN_OPRETURN, &rc );
             plugin_call_plugins( pb, SLAPI_PLUGIN_POST_BIND_FN );
+        } else {
+            /* even though preop failed, we should still call the post-op plugins */
+            plugin_call_plugins( pb, SLAPI_PLUGIN_POST_BIND_FN );
         }
     } else {
         send_ldap_result( pb, LDAP_UNWILLING_TO_PERFORM, NULL,