doPostSearchPluginFNs() should return void for clarity; postoperation

plugins have no mechanism by which they may signal an error condition
to the client.

diff --git a/servers/slapd/search.c b/servers/slapd/search.c
index be1bf7cba9c281adf1ac22cd937c19b568fdd788..c331ed45835082207ba85b7ca9551468ecd2818b 100644 (file)
--- a/servers/slapd/search.c
+++ b/servers/slapd/search.c
@@ -35,7 +35,7 @@ static Slapi_PBlock *initSearchPlugin( Backend *be, Connection *conn, Operation
        int attrsonly, int managedsait );
 static int doPreSearchPluginFNs( Backend *be, Slapi_PBlock *pb );
 static int doSearchRewriteFNs( Backend *be, Slapi_PBlock *pb, Filter **filter, struct berval *fstr );
-static int doPostSearchPluginFNs( Backend *be, Slapi_PBlock *pb );
+static void doPostSearchPluginFNs( Backend *be, Slapi_PBlock *pb );
 #endif /* LDAPI_SLAPI */
 
 int
@@ -525,7 +525,7 @@ static int doSearchRewriteFNs( Backend *be, Slapi_PBlock *pb, Filter **filter, s
        return LDAP_SUCCESS;
 }
 
-static int doPostSearchPluginFNs( Backend *be, Slapi_PBlock *pb )
+static void doPostSearchPluginFNs( Backend *be, Slapi_PBlock *pb )
 {
        if ( doPluginFNs( be, SLAPI_PLUGIN_POST_SEARCH_FN, pb ) != 0 ) {
 #ifdef NEW_LOGGING
@@ -536,8 +536,6 @@ static int doPostSearchPluginFNs( Backend *be, Slapi_PBlock *pb )
                                "failed.\n", 0, 0, 0);
 #endif
        }
-
-       return LDAP_SUCCESS;
 }
 
 void dummy(void)