Added freeing a successful req so it doesnt croud the ldb context

diff --git a/source4/ldap_server/ldap_backend.c b/source4/ldap_server/ldap_backend.c
index d983a542619d47cdb3fa550f2d2cc78a87933e6b..689fd31141466201d63405e976d009b164cee0db 100644 (file)
--- a/source4/ldap_server/ldap_backend.c
+++ b/source4/ldap_server/ldap_backend.c
@@ -206,9 +206,11 @@ static int ldb_mod_req_with_controls(struct ldb_context *ldb,
        }
 
        if (ret == LDB_SUCCESS) {
-               return ldb_transaction_commit(ldb);
+               ret = ldb_transaction_commit(ldb);
+       }
+       else {
+               ldb_transaction_cancel(ldb);
        }
-       ldb_transaction_cancel(ldb);
 
        talloc_free(req);
        return ret;