Uploaded image for project: 'Core ReactOS'
  1. Core ReactOS
  2. CORE-13420

Crash in advapi32_winetest:security test_GetExplicitEntriesFromAclW due to unsupported "CURRENT_USER" trustee

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Fix Version/s: 0.4.6
    • Component/s: RosDlls
    • Labels:

      Description

      https://reactos.org/testman/compare.php?ids=53339,53341,53342,53344

      The crash happens because SetEntriesInAclW fails for the CURRENT_USER trustee, causing new_acl to be null. This causes pGetExplicitEntriesFromAclW to also fail, making access2 NULL:

      7158     access.Trustee.TrusteeForm = TRUSTEE_IS_NAME;
      7159     access.Trustee.ptstrName = (LPWSTR)wszCurrentUser;
      7160     res = pSetEntriesInAclW(1, &access, old_acl, &new_acl);
      7161     ok(res == ERROR_SUCCESS, "SetEntriesInAclW failed: %u\n", res);
      7162     ok(new_acl != NULL, "returned acl was NULL\n");
      7163 
      7164     access2 = NULL;
      7165     res = pGetExplicitEntriesFromAclW(new_acl, &count, &access2);
      7166     ok(res == ERROR_SUCCESS, "GetExplicitEntriesFromAclW failed with error %d\n", GetLastError());
      7167     ok(count == 2, "Expected count == 2, got %d\n", count);
      7168     ok(access2[0].grfAccessMode == GRANT_ACCESS, "Expected GRANT_ACCESS, got %d\n", access2[0].grfAccessMode); //<---- crash happens here

        Attachments

          Issue Links

            Activity

              People

              • Assignee:
                ThFabba ThFabba
                Reporter:
                ThFabba ThFabba
              • Votes:
                0 Vote for this issue
                Watchers:
                0 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: