March 2024 - Wine-gitlab - list.winehq.org

Re: [PATCH v4 0/1] MR1060: include: Remove interfaces already define in msxml6.idl
by Alistair Leslie-Hughes (＠alesliehughes) 16 Mar '24

16 Mar '24

First brings the headers inline with the SDK. Second allows you to build against the wine headers without getting a link time GUID duplicates. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/1060#note_65056

1 0

Re: [PATCH v12 0/3] MR414: fltmgr.sys: Implement FltBuildDefaultSecurityDescriptor
by Zebediah Figura (＠zfigura) 16 Mar '24

16 Mar '24

Zebediah Figura (@zfigura) commented about dlls/ntoskrnl.exe/tests/driver.c: > + PSID group = NULL, owner = NULL; > + BOOLEAN isdefault, present; > + PACL acl = NULL; > + PACCESS_ALLOWED_ACE ace; > + SID_IDENTIFIER_AUTHORITY auth = { SECURITY_NULL_SID_AUTHORITY }; > + SID_IDENTIFIER_AUTHORITY authwine7 = { SECURITY_NT_AUTHORITY }; > + PSID sid1, sid2, sidwin7; > + BOOL ret; > + > + status = FltBuildDefaultSecurityDescriptor(&sd, STANDARD_RIGHTS_ALL); > + ok(status == STATUS_SUCCESS, "got %#lx\n", status); > + if (status != STATUS_SUCCESS) > + { > + win_skip("Skipping FltBuildDefaultSecurityDescriptor tests\n"); > + return; > + } Do we need this? -- https://gitlab.winehq.org/wine/wine/-/merge_requests/414#note_65053

1 0

Re: [PATCH v12 0/3] MR414: fltmgr.sys: Implement FltBuildDefaultSecurityDescriptor
by Zebediah Figura (＠zfigura) 16 Mar '24

16 Mar '24

Zebediah Figura (@zfigura) commented about dlls/ntoskrnl.exe/tests/driver.c: > + } > + ok(sd != NULL, "Failed to return descriptor\n"); > + > + status = RtlGetGroupSecurityDescriptor(sd, &group, &isdefault); > + ok(status == STATUS_SUCCESS, "got %#lx\n", status); > + ok(group == NULL, "group isn't NULL\n"); > + > + status = RtlGetOwnerSecurityDescriptor(sd, &owner, &isdefault); > + ok(status == STATUS_SUCCESS, "got %#lx\n", status); > + ok(owner == NULL, "owner isn't NULL\n"); > + > + status = RtlGetDaclSecurityDescriptor(sd, &present, &acl, &isdefault); > + ok(status == STATUS_SUCCESS, "got %#lx\n", status); > + ok(acl != NULL, "acl is NULL\n"); > + ok(acl->AceCount == 2, "got %d\n", acl->AceCount); > + Should we test "isdefault" here as well? -- https://gitlab.winehq.org/wine/wine/-/merge_requests/414#note_65054

1 0

Re: [PATCH v12 0/3] MR414: fltmgr.sys: Implement FltBuildDefaultSecurityDescriptor
by Zebediah Figura (＠zfigura) 16 Mar '24

16 Mar '24

Zebediah Figura (@zfigura) commented about dlls/ntoskrnl.exe/tests/driver.c: > + *RtlSubAuthoritySid(sidwin7, 1) = DOMAIN_ALIAS_RID_ADMINS; > + > + sid2 = ExAllocatePool(NonPagedPool, RtlLengthRequiredSid(1)); > + RtlInitializeSid(sid2, &auth, 1); > + *RtlSubAuthoritySid(sid2, 0) = SECURITY_LOCAL_SYSTEM_RID; > + > + /* SECURITY_BUILTIN_DOMAIN_RID */ > + status = RtlGetAce(acl, 0, (void**)&ace); > + ok(status == STATUS_SUCCESS, "got %#lx\n", status); > + > + ok(ace->Header.AceType == ACCESS_ALLOWED_ACE_TYPE, "got %#x\n", ace->Header.AceType); > + ok(ace->Header.AceFlags == 0, "got %#x\n", ace->Header.AceFlags); > + ok(ace->Mask == STANDARD_RIGHTS_ALL, "got %#lx\n", ace->Mask); > + > + ret = RtlEqualSid(sid1, (PSID)&ace->SidStart) || RtlEqualSid(sidwin7, (PSID)&ace->SidStart); > + ok(ret, "SID not equal\n"); Maybe it'd be a bit simpler to test the SID elements individually rather than constructing a SID and using RtlEqualSid()? -- https://gitlab.winehq.org/wine/wine/-/merge_requests/414#note_65055

1 0

Re: [PATCH v12 0/3] MR414: fltmgr.sys: Implement FltBuildDefaultSecurityDescriptor
by Zebediah Figura (＠zfigura) 16 Mar '24

16 Mar '24

Zebediah Figura (@zfigura) commented about dlls/fltmgr.sys/main.c: > + goto done; > + RtlInitializeSid(sid, &auth, 2); > + sid->SubAuthority[1] = DOMAIN_GROUP_RID_ADMINS; > + sid->SubAuthority[0] = SECURITY_BUILTIN_DOMAIN_RID; > + > + sid_len = RtlLengthRequiredSid(1); > + sid_system = ExAllocatePool(PagedPool, sid_len); > + if (!sid_system) > + goto done; > + RtlInitializeSid(sid_system, &auth, 1); > + sid_system->SubAuthority[0] = SECURITY_LOCAL_SYSTEM_RID; > + > + sid_len = SECURITY_DESCRIPTOR_MIN_LENGTH + sizeof(ACL) + > + sizeof(ACCESS_ALLOWED_ACE) + RtlLengthSid(sid) + > + sizeof(ACCESS_ALLOWED_ACE) + RtlLengthSid(sid_system); > + Maybe rename "sid_len" to "len" if you're going to use it for other things? -- https://gitlab.winehq.org/wine/wine/-/merge_requests/414#note_65052

1 0

Re: [PATCH v12 0/3] MR414: fltmgr.sys: Implement FltBuildDefaultSecurityDescriptor
by Zebediah Figura (＠zfigura) 16 Mar '24

16 Mar '24

Zebediah Figura (@zfigura) commented about dlls/fltmgr.sys/main.c: > > return func; > } > + > +NTSTATUS WINAPI FltBuildDefaultSecurityDescriptor(PSECURITY_DESCRIPTOR *descriptor, ACCESS_MASK access) > +{ > + PACL dacl; > + NTSTATUS ret = STATUS_INSUFFICIENT_RESOURCES; This initialization isn't doing anything. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/414#note_65051

1 0

Re: [PATCH v4 0/1] MR1060: include: Remove interfaces already define in msxml6.idl
by Nikolay Sivov (＠nsivov) 16 Mar '24

16 Mar '24

@alesliehughes could you remind me what this was going to fix? -- https://gitlab.winehq.org/wine/wine/-/merge_requests/1060#note_65050

1 0

[PATCH 0/1] MR1228: wined3d: support direct3d shader model 1,2,3
by Fan WenJie (＠fanwenjie) 16 Mar '24

16 Mar '24

vkd3d_shader_compile_info::source_type can not hard write to VKD3D_SHADER_SOURCE_DXBC_TPF, it is decided by magic number of shader source. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/1228

4 5

[PATCH 0/1] MR5332: user32/tests: Accept WM_ACTIVATE in TrackPopupMenu.
by Esme Povirk (＠madewokherd) 16 Mar '24

16 Mar '24

https://test.winehq.org/data/552cc456d1889ab3ee0dd5ead6c7520c34d030c0/w1121… -- https://gitlab.winehq.org/wine/wine/-/merge_requests/5332

2 1

[PATCH v3 0/2] MR5247: user32/msgbox: Support WM_COPY Message
by Alistair Leslie-Hughes (＠alesliehughes) 16 Mar '24

16 Mar '24

Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=17205 -- v3: user32/msgbox: Use a windows hook to trap Ctrl+C user32/msgbox: Support WM_COPY Message https://gitlab.winehq.org/wine/wine/-/merge_requests/5247

4 9