From 23a95931d3b868c6e6bcda0c214be6a12434ae8b Mon Sep 17 00:00:00 2001 From: David Disseldorp Date: Mon, 26 May 2014 15:48:34 +0200 Subject: [PATCH] libcli/secdesc: replace dup_sec_desc() usage Use security_descriptor_copy() instead, which is also provided by libcli. Signed-off-by: David Disseldorp Reviewed-by: Jeremy Allison Reviewed-by: Volker Lendecke --- libcli/security/secdesc.c | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/libcli/security/secdesc.c b/libcli/security/secdesc.c index 31d134b6e70..72ba2dba589 100644 --- a/libcli/security/secdesc.c +++ b/libcli/security/secdesc.c @@ -405,8 +405,11 @@ struct sec_desc_buf *make_sec_desc_buf(TALLOC_CTX *ctx, size_t len, struct secur /* max buffer size (allocated size) */ dst->sd_size = (uint32_t)len; - if(sec_desc && ((dst->sd = dup_sec_desc(ctx, sec_desc)) == NULL)) { - return NULL; + if (sec_desc != NULL) { + dst->sd = security_descriptor_copy(ctx, sec_desc); + if (dst->sd == NULL) { + return NULL; + } } return dst; -- 2.11.4.GIT