October 2023 - wine-gitlab

Re: [PATCH v4 0/3] MR4168: cmd: Handle caret / blocks with whitespace only lines better
by userid0x0 (＠userid0x0) 27 Oct '23

27 Oct '23

Okay, oh dear. I need to apologize it's the first time I contribute to wine and as always the user learns over time. Now I have the build environment on a Windows 10 Machine set up and I can run the testbench on Windows as well. Cool feature! Lesson learned: I focussed on the unbalanced parentheses case first. Windows quits/ends the complete `cmd` execution if it do a `call block_if_eof1.cmd`. Did you have this case yet? In regards of testing this means I need to setup a own `.cmd`/`.cmd.exp` file again? Lets form a simple setup ``` echo @echo off>block_if_eof1.cmd echo echo sanity1>>block_if_eof1.cmd echo if 1 == 1 (>>block_if_eof1.cmd echo echo should not be executed>>block_if_eof1.cmd call block_if_eof1.cmd REM This command is NEVER executed as cmd exe quited. ``` -- https://gitlab.winehq.org/wine/wine/-/merge_requests/4168#note_49957

1 0

Re: [PATCH v4 0/3] MR4168: cmd: Handle caret / blocks with whitespace only lines better
by userid0x0 (＠userid0x0) 27 Oct '23

27 Oct '23

On Thu Oct 26 17:31:50 2023 +0000, eric pouech wrote: > nitpick: you could have saved one line of modif by keeping the == 0x00 > test (instead of rewritten to 0x0) Can fix that as well. No Problem must have happened while fixup. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/4168#note_49956

1 0

[PATCH v5 0/3] MR4073: msxml3: Stop possible typelib out of bounds access (Coverity)
by Alistair Leslie-Hughes (＠alesliehughes) 27 Oct '23

27 Oct '23

typelib has an array size of 2 (eg LibXml_Last), so a lookup of IID_NULL will result in a lookup of the third index. -- v5: msxml3: Free IBindCtx on error paths (Coverity) msxml3: Move tid_NULL out of possible enum values. msxml3: Dont call qsort if we have no data (Coverity). https://gitlab.winehq.org/wine/wine/-/merge_requests/4073

2 3

[PATCH v4 0/3] MR4073: msxml3: Stop possible typelib out of bounds access (Coverity)
by Alistair Leslie-Hughes (＠alesliehughes) 27 Oct '23

27 Oct '23

typelib has an array size of 2 (eg LibXml_Last), so a lookup of IID_NULL will result in a lookup of the third index. -- v4: msxml3: Free IBindCtx on error paths (Coverity) https://gitlab.winehq.org/wine/wine/-/merge_requests/4073

2 3

Re: [PATCH v3 0/3] MR4073: msxml3: Stop possible typelib out of bounds access (Coverity)
by Nikolay Sivov (＠nsivov) 27 Oct '23

27 Oct '23

On Fri Oct 27 00:52:11 2023 +0000, Nikolay Sivov wrote: > If it's a matter of silencing typelib[LibXml_Last] access warning, just > change it to LibXml2, and add an assert for tid range in get_typeinfo(). Or maybe just change Last -> LibXml2, without asserts. All the calls are internal, there is more value in keeping this looking close to other modules using the same logic, than trying to fix what can't happen. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/4073#note_49942

1 0

Re: [PATCH v3 0/3] MR4073: msxml3: Stop possible typelib out of bounds access (Coverity)
by Nikolay Sivov (＠nsivov) 27 Oct '23

27 Oct '23

Nikolay Sivov (@nsivov) commented about dlls/msxml3/httprequest.c: > if (!(ptr = heap_alloc(size))) > { > heap_free(bsc); > + IBindCtx_Release(pbc); > return E_OUTOFMEMORY; > } It will be a smaller change to move context creation right before it's used. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/4073#note_49937

1 0

Re: [PATCH v3 0/3] MR4073: msxml3: Stop possible typelib out of bounds access (Coverity)
by Nikolay Sivov (＠nsivov) 27 Oct '23

27 Oct '23

On Thu Oct 19 03:42:38 2023 +0000, Alex Henrie wrote: > You could change the first element of tid_ids from { &IID_NULL, > LibXml_Last } to { NULL } to make it more clear that it is never used. > Or, if we can't come to a consensus, you could mark the warning as a > false positive in Coverity with a link to this discussion. If it's a matter of silencing typelib[LibXml_Last] access warning, just change it to LibXml2, and add an assert for tid range in get_typeinfo(). -- https://gitlab.winehq.org/wine/wine/-/merge_requests/4073#note_49936

1 0

[PATCH 0/2] MR4193: ws2_32/tests: Test fixes.
by Zebediah Figura (＠zfigura) 26 Oct '23

26 Oct '23

-- https://gitlab.winehq.org/wine/wine/-/merge_requests/4193

2 2

[PATCH v3 0/9] MR396: vkd3d-shader/hlsl: Support non-constant offset dereferences, v2, part 1.
by Francisco Casas (＠fcasas) 26 Oct '23

26 Oct '23

First part of the continuation of the implementation of non-constant offset dereferences (a.k.a. relative addressing) for SM4, now that we use vsir registers in tpf.c. As a quick recap: while parsing HLSL we are expressing derefs as paths, and then we are lowering these paths into a single offset node (which is closer to the bytecode) using the replace_deref_path_with_offset() pass, right before register allocation. This first part of the series splits this offset node into 2 parts: - A constant uint, which will be called hlsl_deref.offset_const. - A non-hlsl_ir_constant offset node that will only be present when we need relative addressing, that we will end up calling hlsl_deref.offset_rel. Both these fields will be analog to the ones used in vsir register indexes, vkd3d_shader_register_index.rel_addr and vkd3d_shader_register_index.offset respectively, which is something we need for the second part of this series. The following patches are in my [nonconst-offsets-8](https://gitlab.winehq.org/fcasas/vkd3d/-/commits/noncon… branch, if something is not clear in this series, it may be worth skimming through them. Supersedes !229. -- v3: vkd3d-shader/tpf: Declare indexable temps. https://gitlab.winehq.org/wine/vkd3d/-/merge_requests/396

4 13

Re: [PATCH v2 0/1] MR4126: winex11.drv: Send missed KEYUP events on KeymapNotify.
by Rémi Bernon (＠rbernon) 26 Oct '23

26 Oct '23

Sure, we can change the tests, maybe move the child window test down if that's enough. it's just there to check that only top-level windows are accepted for cooperative level. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/4126#note_49931

1 0

2025

2024

2023

2022

wine-gitlab October 2023