]> git.ipfire.org Git - thirdparty/samba.git/commitdiff
s4:auth: Use Anonymous and System SID constants
authorJoseph Sutton <josephsutton@catalyst.net.nz>
Mon, 11 Sep 2023 03:26:41 +0000 (15:26 +1200)
committerAndrew Bartlett <abartlet@samba.org>
Wed, 27 Sep 2023 02:43:28 +0000 (02:43 +0000)
Signed-off-by: Joseph Sutton <josephsutton@catalyst.net.nz>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
source4/auth/session.c

index 9e893ec3e920f15edf87987062934785fa7c3e5f..ba4541bd418578ba7bd1c471f3400a9f81c61afb 100644 (file)
@@ -62,26 +62,12 @@ _PUBLIC_ NTSTATUS auth_generate_security_token(TALLOC_CTX *mem_ctx,
        uint32_t num_sids = 0;
        const char *filter = NULL;
        struct auth_SidAttr *sids = NULL;
-       const struct dom_sid *anonymous_sid = NULL;
-       const struct dom_sid *system_sid = NULL;
 
        TALLOC_CTX *tmp_ctx = talloc_new(mem_ctx);
        if (tmp_ctx == NULL) {
                return NT_STATUS_NO_MEMORY;
        }
 
-       anonymous_sid = dom_sid_parse_talloc(tmp_ctx, SID_NT_ANONYMOUS);
-       if (anonymous_sid == NULL) {
-               TALLOC_FREE(tmp_ctx);
-               return NT_STATUS_NO_MEMORY;
-       }
-
-       system_sid = dom_sid_parse_talloc(tmp_ctx, SID_NT_SYSTEM);
-       if (system_sid == NULL) {
-               TALLOC_FREE(tmp_ctx);
-               return NT_STATUS_NO_MEMORY;
-       }
-
        sids = talloc_array(tmp_ctx, struct auth_SidAttr, user_info_dc->num_sids);
        if (sids == NULL) {
                TALLOC_FREE(tmp_ctx);
@@ -144,9 +130,9 @@ _PUBLIC_ NTSTATUS auth_generate_security_token(TALLOC_CTX *mem_ctx,
        }
 
 
-       if (num_sids > PRIMARY_USER_SID_INDEX && dom_sid_equal(anonymous_sid, &sids[PRIMARY_USER_SID_INDEX].sid)) {
+       if (num_sids > PRIMARY_USER_SID_INDEX && dom_sid_equal(&global_sid_Anonymous, &sids[PRIMARY_USER_SID_INDEX].sid)) {
                /* Don't expand nested groups of system, anonymous etc*/
-       } else if (num_sids > PRIMARY_USER_SID_INDEX && dom_sid_equal(system_sid, &sids[PRIMARY_USER_SID_INDEX].sid)) {
+       } else if (num_sids > PRIMARY_USER_SID_INDEX && dom_sid_equal(&global_sid_System, &sids[PRIMARY_USER_SID_INDEX].sid)) {
                /* Don't expand nested groups of system, anonymous etc*/
        } else if (sam_ctx != NULL) {
                filter = talloc_asprintf(tmp_ctx, "(&(objectClass=group)(groupType:"LDB_OID_COMPARATOR_AND":=%u))",