Possible missing send_ldap_result in ldbm_back_modify 1.2.7

[Pierangelo Masarati <pmasarati@bci.it>]

Hello.

I noticed, by intentionally forcing an error in ldbm_internal_modify, an
internal
of back-ldbm in OpenLdap-1.2.7, that the client hangs waiting for a
reply.
I guess this is due to ldbm_back_modify that in case of such failure
sends to error_return where no send_ldap_result appears.


        /* Modify the entry */
        if ( ldbm_internal_modify( be, conn, op, dn, mods, e ) != 0 ) {

                goto error_return;

        }

error_return:;
        cache_return_entry_w( &li->li_cache, e );
        return( -1 );
}





The fix depends on whether and how the exception is to be handled, but I
guess
somewhere in the code a send_ldap_result raising the error should be
placed.

Regards,

Pierangelo Masarati

http://www.sys-net.it
mailto:ando@sys-net.it