summaryrefslogtreecommitdiff
path: root/lib/ldb
diff options
context:
space:
mode:
authorDouglas Bagnall <douglas.bagnall@catalyst.net.nz>2016-03-11 10:26:10 +1300
committerAndrew Bartlett <abartlet@samba.org>2016-03-22 08:00:31 +0100
commit578643fb3613c6c59e31d848a032fea2fd514dd5 (patch)
tree7196ce6cbfb290fd0449ce30dabe560ef20025eb /lib/ldb
parent79c58c2543790134ef8deb5307ec4b4ae069ea61 (diff)
downloadsamba-578643fb3613c6c59e31d848a032fea2fd514dd5.tar.gz
ldb controls: don't ignore memory allocation failure
Thanks to Jeremy Allison for noticing this. Signed-off-by: Douglas Bagnall <douglas.bagnall@catalyst.net.nz> Reviewed-by: Andrew Bartlett <abartlet@samba.org> Reviewed-by: Garming Sam <garming@catalyst.net.nz>
Diffstat (limited to 'lib/ldb')
-rw-r--r--lib/ldb/common/ldb_controls.c3
1 files changed, 3 insertions, 0 deletions
diff --git a/lib/ldb/common/ldb_controls.c b/lib/ldb/common/ldb_controls.c
index 0fdd13abc65..ba813ea04cf 100644
--- a/lib/ldb/common/ldb_controls.c
+++ b/lib/ldb/common/ldb_controls.c
@@ -315,6 +315,9 @@ char *ldb_control_to_string(TALLOC_CTX *mem_ctx, const struct ldb_control *contr
cookie = ldb_base64_encode(mem_ctx,
(char *)rep_control->contextId,
rep_control->ctxid_len);
+ if (cookie == NULL) {
+ return NULL;
+ }
res = talloc_asprintf(mem_ctx, "%s:%d:%d:%d:%d:%s",
LDB_CONTROL_VLV_RESP_NAME,