libcli/security: Make use of sids_contains_sid_attrs()

Signed-off-by: Joseph Sutton <josephsutton@catalyst.net.nz>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>

diff --git a/libcli/security/util_sid.c b/libcli/security/util_sid.c
index 33886e788f70aaaf144ca982125f7712d38cd908..b3408b37a4690942b857ba8aaa84363d4f4d235c 100644 (file)
--- a/libcli/security/util_sid.c
+++ b/libcli/security/util_sid.c
@@ -444,16 +444,10 @@ NTSTATUS add_sid_to_array_attrs_unique(TALLOC_CTX *mem_ctx,
                                       const struct dom_sid *sid, uint32_t attrs,
                                       struct auth_SidAttr **sids, uint32_t *num_sids)
 {
-       uint32_t i;
-
-       for (i=0; i<(*num_sids); i++) {
-               if (attrs != (*sids)[i].attrs) {
-                       continue;
-               }
-               if (!dom_sid_equal(sid, &(*sids)[i].sid)) {
-                       continue;
-               }
+       bool contains;
 
+       contains = sids_contains_sid_attrs(*sids, *num_sids, sid, attrs);
+       if (contains) {
                return NT_STATUS_OK;
        }