]> git.ipfire.org Git - thirdparty/samba.git/commitdiff
libcli: Simplify dom_sid_dup()
authorVolker Lendecke <vl@samba.org>
Tue, 18 Apr 2023 11:39:09 +0000 (13:39 +0200)
committerJeremy Allison <jra@samba.org>
Fri, 16 Jun 2023 16:14:30 +0000 (16:14 +0000)
We have sid_copy() for this.

Signed-off-by: Volker Lendecke <vl@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
libcli/security/dom_sid.c

index 52d4454e4e59a4ea88bd337ad9f6a74fedbec7c1..9a91760ff6246d20f14a34da29b17ec3d7189357 100644 (file)
@@ -290,7 +290,6 @@ struct dom_sid *dom_sid_parse_length(TALLOC_CTX *mem_ctx, const DATA_BLOB *sid)
 struct dom_sid *dom_sid_dup(TALLOC_CTX *mem_ctx, const struct dom_sid *dom_sid)
 {
        struct dom_sid *ret;
-       int i;
 
        if (!dom_sid) {
                return NULL;
@@ -300,19 +299,7 @@ struct dom_sid *dom_sid_dup(TALLOC_CTX *mem_ctx, const struct dom_sid *dom_sid)
        if (!ret) {
                return NULL;
        }
-
-       ret->sid_rev_num = dom_sid->sid_rev_num;
-       ret->id_auth[0] = dom_sid->id_auth[0];
-       ret->id_auth[1] = dom_sid->id_auth[1];
-       ret->id_auth[2] = dom_sid->id_auth[2];
-       ret->id_auth[3] = dom_sid->id_auth[3];
-       ret->id_auth[4] = dom_sid->id_auth[4];
-       ret->id_auth[5] = dom_sid->id_auth[5];
-       ret->num_auths = dom_sid->num_auths;
-
-       for (i=0;i<dom_sid->num_auths;i++) {
-               ret->sub_auths[i] = dom_sid->sub_auths[i];
-       }
+       sid_copy(ret, dom_sid);
 
        return ret;
 }