diff options
author | Andrew Bartlett <abartlet@samba.org> | 2007-04-23 07:22:16 +0000 |
---|---|---|
committer | Gerald (Jerry) Carter <jerry@samba.org> | 2007-10-10 14:51:30 -0500 |
commit | f72d4bfea1d436712448619b257930aa52e543c2 (patch) | |
tree | d639dbfbc514421b790e9238edf5cf6a464b9d47 /source4 | |
parent | 023e2451873a89a433fadbe913b94507af856cf3 (diff) | |
download | samba-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)
Diffstat (limited to 'source4')
-rw-r--r-- | source4/lib/ldb/common/ldb.c | 2 |
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); |