diff options
author | Douglas Bagnall <douglas.bagnall@catalyst.net.nz> | 2016-03-11 10:26:10 +1300 |
---|---|---|
committer | Andrew Bartlett <abartlet@samba.org> | 2016-03-22 08:00:31 +0100 |
commit | 578643fb3613c6c59e31d848a032fea2fd514dd5 (patch) | |
tree | 7196ce6cbfb290fd0449ce30dabe560ef20025eb /lib/ldb | |
parent | 79c58c2543790134ef8deb5307ec4b4ae069ea61 (diff) | |
download | samba-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.c | 3 |
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, |