summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorAndrew Bartlett <abartlet@samba.org>2007-04-23 07:22:16 +0000
committerGerald (Jerry) Carter <jerry@samba.org>2007-10-10 14:51:30 -0500
commitf72d4bfea1d436712448619b257930aa52e543c2 (patch)
treed639dbfbc514421b790e9238edf5cf6a464b9d47
parent023e2451873a89a433fadbe913b94507af856cf3 (diff)
downloadsamba-f72d4bfea1d436712448619b257930aa52e543c2.tar.gz
samba-f72d4bfea1d436712448619b257930aa52e543c2.tar.bz2
samba-f72d4bfea1d436712448619b257930aa52e543c2.zip
r22474: If ldb does not return sucess, then the res variable may not be valid.
It *should* just be NULL from the initialisation above, but I've had this not be the case... Andrew Bartlett (This used to be commit a2848dbf1fa9eafdef93bd914b12ff2f9f223a70)
-rw-r--r--source4/lib/ldb/common/ldb.c2
1 files changed, 0 insertions, 2 deletions
diff --git a/source4/lib/ldb/common/ldb.c b/source4/lib/ldb/common/ldb.c
index 5b3520766d..c317fe57fa 100644
--- a/source4/lib/ldb/common/ldb.c
+++ b/source4/lib/ldb/common/ldb.c
@@ -970,8 +970,6 @@ int ldb_search_exp_fmt(struct ldb_context *ldb, TALLOC_CTX *mem_ctx, struct ldb_
if (ret == LDB_SUCCESS) {
talloc_steal(mem_ctx, res);
*result = res;
- } else {
- talloc_free(res);
}
talloc_free(expression);