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

Re: [PATCH 0/1] MR5727: riched20: In EM_SETPARAFORMAT protect against out of bound cTabStop values
by Nikolay Sivov (＠nsivov) 28 May '24

28 May '24

Nikolay Sivov (@nsivov) commented about dlls/riched20/editor.c: > } > case EM_SETPARAFORMAT: > { > - BOOL result = editor_set_selection_para_fmt( editor, (PARAFORMAT2 *)lParam ); > + BOOL result; > + PARAFORMAT2 fmt = *(PARAFORMAT2 *)lParam; > + fmt.cTabCount = max(0, min(fmt.cTabCount, 32)); /* Clamp between 0 and 32 */ > + result = editor_set_selection_para_fmt(editor, &fmt); I think this is would be better if done in para_set_fmt(). Instead of 32 please use MAX_TAB_STOPS or ARRAY_SIZE(). -- https://gitlab.winehq.org/wine/wine/-/merge_requests/5727#note_71380

1 0

Re: [PATCH 0/1] MR5727: riched20: In EM_SETPARAFORMAT protect against out of bound cTabStop values
by Nikolay Sivov (＠nsivov) 28 May '24

28 May '24

Nikolay Sivov (@nsivov) commented about dlls/riched20/tests/editor.c: > SendMessageW(richedit, EM_GETPARAFORMAT, 0, (LPARAM)&pf); > ok(pf.wAlignment == align_mask[i], "got %d expect %ld\n", pf.wAlignment, align_mask[i]); > > + /* Test out of bounds tab count */ > + pf.cTabCount = -25000; > + SendMessageW(richedit, EM_SETPARAFORMAT, 0, (LPARAM)&pf); > + ok(pf.cTabCount == -25000, "Got %d\n", pf.cTabCount); > + pf.cTabCount = 25000; > + SendMessageW(richedit, EM_SETPARAFORMAT, 0, (LPARAM)&pf); > + ok(pf.cTabCount == 25000, "Got %d\n", pf.cTabCount); It makes more sense to test EM_GET* results here. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/5727#note_71379

1 0

Re: [PATCH v2 0/1] MR5721: wow64: In wow64_NtSetInformationToken forward TokenIntegrityLevel
by Fabian Maurer (＠DarkShadow44) 28 May '24

28 May '24

On Mon May 27 23:57:31 2024 +0000, Jinoh Kang wrote: > It's worth noting that we already have multiple > `STATUS_SUCCESS`-returning WoW64 stubs: > - `wow64_NtSetInformationProcess(ProcessInstrumentationCallback)`, at > dlls/wow64/process.c:896 ([gitlab][gitlab-sip], [cgit][cgit-sip]) > - `wow64_NtQueryMultipleValueKey`, at dlls/wow64/registry.c:364 > ([gitlab][gitlab-mvk], [cgit][cgit-mvk]) > - `wow64_NtCreateLowBoxToken`, at dlls/wow64/security.c:134 > ([gitlab][gitlab-clbt], [cgit][cgit-clbt]) > - `wow64_NtSetInformationJobObject(JobObjectBasicUIRestrictions)`, at > dlls/wow64/sync.c:1331 ([gitlab][gitlab-sijo], [cgit][cgit-sijo]) > A possible concern is that someone might introduce inconsistency in the > future, by forgetting to update the WoW64 stub when the syscall itself > is implemented. > Fortunately, I'm optimistic that we'll start testing wow64 regularly > before it becomes the new default, so maybe we just need a test in > `advapi32:security` to catch such oversight leading to inconsistency. > (That said, many of the above need proper tests too.) > [gitlab-sip]: <https://gitlab.winehq.org/wine/wine/-/blob/b210a204137dec8d2126ca909d762454…> > [cgit-sip]: <https://source.winehq.org/git/wine.git/blob/b210a204137dec8d2126ca909d76245…> > [gitlab-mvk]: <https://gitlab.winehq.org/wine/wine/-/blob/b210a204137dec8d2126ca909d762454…> > [cgit-mvk]: <https://source.winehq.org/git/wine.git/blob/b210a204137dec8d2126ca909d76245…> > [gitlab-clbt]: <https://gitlab.winehq.org/wine/wine/-/blob/b210a204137dec8d2126ca909d762454…> > [cgit-clbt]: <https://source.winehq.org/git/wine.git/blob/b210a204137dec8d2126ca909d76245…> > [gitlab-sijo]: <https://gitlab.winehq.org/wine/wine/-/blob/b210a204137dec8d2126ca909d762454…> > [cgit-sijo]: <https://source.winehq.org/git/wine.git/blob/b210a204137dec8d2126ca909d76245…> Alright, updated -- https://gitlab.winehq.org/wine/wine/-/merge_requests/5721#note_71344

1 0

Re: [PATCH 0/1] MR5721: wow64: In wow64_NtSetInformationToken forward TokenIntegrityLevel
by Jinoh Kang (＠iamahuman) 28 May '24

28 May '24

On Mon May 27 23:57:31 2024 +0000, Nikolay Sivov wrote: > This should forward to an actual syscall, instead of having stubs in > both places. It's worth noting that we already have multiple `STATUS_SUCCESS`-returning WoW64 stubs: - `wow64_NtSetInformationProcess(ProcessInstrumentationCallback)`, at dlls/wow64/process.c:896 ([gitlab][gitlab-sip], [cgit][cgit-sip]) - `wow64_NtQueryMultipleValueKey`, at dlls/wow64/registry.c:364 ([gitlab][gitlab-mvk], [cgit][cgit-mvk]) - `wow64_NtCreateLowBoxToken`, at dlls/wow64/security.c:134 ([gitlab][gitlab-clbt], [cgit][cgit-clbt]) - `wow64_NtSetInformationJobObject(JobObjectBasicUIRestrictions)`, at dlls/wow64/sync.c:1331 ([gitlab][gitlab-sijo], [cgit][cgit-sijo]) A possible concern is that someone might introduce inconsistency in the future, by forgetting to update the WoW64 stub when the syscall itself is implemented. Fortunately, I'm optimistic that we'll start testing wow64 regularly before it becomes the new default, so maybe we just need a test in `advapi32:security` to catch such oversight leading to inconsistency. (That said, many of the above need proper tests too.) [gitlab-sip]: <https://gitlab.winehq.org/wine/wine/-/blob/b210a204137dec8d2126ca909d762454…> [cgit-sip]: <https://source.winehq.org/git/wine.git/blob/b210a204137dec8d2126ca909d76245…> [gitlab-mvk]: <https://gitlab.winehq.org/wine/wine/-/blob/b210a204137dec8d2126ca909d762454…> [cgit-mvk]: <https://source.winehq.org/git/wine.git/blob/b210a204137dec8d2126ca909d76245…> [gitlab-clbt]: <https://gitlab.winehq.org/wine/wine/-/blob/b210a204137dec8d2126ca909d762454…> [cgit-clbt]: <https://source.winehq.org/git/wine.git/blob/b210a204137dec8d2126ca909d76245…> [gitlab-sijo]: <https://gitlab.winehq.org/wine/wine/-/blob/b210a204137dec8d2126ca909d762454…> [cgit-sijo]: <https://source.winehq.org/git/wine.git/blob/b210a204137dec8d2126ca909d76245…> -- https://gitlab.winehq.org/wine/wine/-/merge_requests/5721#note_71333

1 0

Re: [PATCH 0/1] MR5721: wow64: In wow64_NtSetInformationToken forward TokenIntegrityLevel
by Nikolay Sivov (＠nsivov) 28 May '24

28 May '24

This should forward to an actual syscall, instead of having stubs in both places. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/5721#note_71332

1 0

Re: [PATCH v2 0/4] MR5667: winegstreamer: Use GstCaps for wg_parser internal formats.
by Rémi Bernon 28 May '24

28 May '24

I'm sorry but I've wasted enough time on this. I believe I'm still leaving more than enough time to give some feedback, either commenting on the code or even rejecting the changes for whatever reason you might find appropriate. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/5667#note_71331

1 0

Re: [PATCH v2 0/4] MR5667: winegstreamer: Use GstCaps for wg_parser internal formats.
by Elizabeth Figura (＠zfigura) 28 May '24

28 May '24

> * [Rémi Bernon](/rbernon) removed review request for @zfigura [<time datetime="2024-05-24T07:01:08.204Z">May 24, 2024, 2:01 AM</time>](#note_71260) Please don't do that. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/5667#note_71330

1 0

Re: [PATCH v2 0/2] MR5708: user32/tests: Rename and expand use of winevent_hook_todo flag. - approved
by Alexandre Julliard (＠julliard) 28 May '24

28 May '24

This merge request was approved by Alexandre Julliard. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/5708

1 0

Re: [PATCH v2 0/1] MR5718: shell32/tests: fixed sizeof to GetModuleFileName - approved
by Alexandre Julliard (＠julliard) 28 May '24

28 May '24

This merge request was approved by Alexandre Julliard. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/5718

1 0

Re: [PATCH v2 0/1] MR5715: gitlab: Use --enable-werror for Clang builds. - approved
by Alexandre Julliard (＠julliard) 28 May '24

28 May '24

This merge request was approved by Alexandre Julliard. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/5715

1 0