svn commit: samba r17303 - in branches/SAMBA_4_0/source/dsdb/samdb/ldb_modules: .

abartlet at samba.org abartlet at samba.org
Sat Jul 29 02:00:35 GMT 2006


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);



More information about the samba-cvs mailing list