Author: abartlet
Date: 2006-07-29 02:00:33 +0000 (Sat, 29 Jul 2006)
New Revision: 17303

WebSVN: 
http://websvn.samba.org/cgi-bin/viewcvs.cgi?view=rev&root=samba&rev=17303

Log:
More testing results: Don't try and call a NULL callback, and use the
correct parameter, as this is called for more than just 'add'.

Andrew Bartlett

Modified:
   branches/SAMBA_4_0/source/dsdb/samdb/ldb_modules/partition.c


Changeset:
Modified: branches/SAMBA_4_0/source/dsdb/samdb/ldb_modules/partition.c
===================================================================
--- branches/SAMBA_4_0/source/dsdb/samdb/ldb_modules/partition.c        
2006-07-29 01:52:15 UTC (rev 17302)
+++ branches/SAMBA_4_0/source/dsdb/samdb/ldb_modules/partition.c        
2006-07-29 02:00:33 UTC (rev 17303)
@@ -164,6 +164,11 @@
 
        ac = talloc_get_type(context, struct partition_context);
 
+       if (!ac->orig_req->callback) {
+               talloc_free(ares);
+               return LDB_SUCCESS;
+       }
+
        if (!ares 
            || (ares->type == LDB_REPLY_EXTENDED 
                && strcmp(ares->response->oid, LDB_EXTENDED_START_TLS_OID))) {
@@ -271,7 +276,7 @@
        /* Otherwise, we need to find the backend to fire it to */
 
        /* Find backend */
-       backend = find_backend(module, req, req->op.add.message->dn);
+       backend = find_backend(module, req, dn);
        
        /* issue request */
        return ldb_next_request(backend, req);

Reply via email to