February 2023 - Wine-gitlab

[PATCH v22 0/2] MR2100: ntdll: Do not send data to port 0.
by Jason Beetham (＠beef331) 14 Feb '23

14 Feb '23

On Windows it seems sending to port 0 does nothing and does not error. Presently sendmsg errors with EINVAL. This works around it, by checking if it's port 0 then skipping the data. -- v22: ntdll: Do not send data to port 0. https://gitlab.winehq.org/wine/wine/-/merge_requests/2100

2 2

[PATCH v9 0/2] MR2175: oleaut32: fix VarAbs function for BSTR with positive values
by Jason Millard (＠jsm174) 14 Feb '23

14 Feb '23

While working in vbscript, I noticed abs on a `BSTR` value with a positive number would come back with an invalid value. Given the following script: ``` Dim x x = "30000" Debug.Print "x" & "=" & abs(x) ``` Outputs: ``` Script.Print 'x=3.08221696253945E-314' ``` After debugging, `pVarIn` gets the R8 value but is never shallow copied into `pVarOut`. In addition to a test case in `oleaut32`, I've included a test case in `vbscript`. This fixes bug: https://bugs.winehq.org/show_bug.cgi?id=54489 -- v9: oleaut32: fix VarAbs function for BSTR with positive values https://gitlab.winehq.org/wine/wine/-/merge_requests/2175

4 3

Re: [PATCH v21 0/2] MR2100: ntdll: Do not send data to port 0.
by Zebediah Figura (＠zfigura) 14 Feb '23

14 Feb '23

Thanks, looks good now, I just have a couple more tweaks to keep the style consistent. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/2100#note_24098

1 0

Re: [PATCH v21 0/2] MR2100: ntdll: Do not send data to port 0.
by Zebediah Figura (＠zfigura) 14 Feb '23

14 Feb '23

Zebediah Figura (@zfigura) commented about dlls/ntdll/unix/socket.c: > { > ULONG_PTR information; > > - status = try_send( fd, async ); > + if (protocol == WS_IPPROTO_UDP && async->addr && sockaddr_is_port0(async->addr, async->addr_len)) > + { > + /* Spellforce 3 is known to send to port 0. > + * This causes 'sendmsg' to throw a EINVAL error, on Windows this does nothing but consume the data. > + */ > + ssize_t i; > + for(i = async->iov_cursor; i < async->count; i++) > + { > + async->sent_len += async->iov[i].iov_len; > + } ```suggestion:-3+0 for (i = async->iov_cursor; i < async->count; i++) async->sent_len += async->iov[i].iov_len; ``` -- https://gitlab.winehq.org/wine/wine/-/merge_requests/2100#note_24096

1 0

Re: [PATCH v21 0/2] MR2100: ntdll: Do not send data to port 0.
by Zebediah Figura (＠zfigura) 14 Feb '23

14 Feb '23

Zebediah Figura (@zfigura) commented about dlls/ntdll/unix/socket.c: > + * This causes 'sendmsg' to throw a EINVAL error, on Windows this does nothing but consume the data. > + */ > + ssize_t i; > + for(i = async->iov_cursor; i < async->count; i++) > + { > + async->sent_len += async->iov[i].iov_len; > + } > + > + WARN("Attempting to send to port 0, skipping over data.\n"); > + > + status = STATUS_SUCCESS; > + } > + else > + { > + status = try_send( fd, async ); > + } ```suggestion:-2+0 { status = try_send( fd, async ); } ``` -- https://gitlab.winehq.org/wine/wine/-/merge_requests/2100#note_24097

1 0

[PATCH v11 0/8] MR1628: ntdll: Implement Low Fragmentation Heap.
by Rémi Bernon 14 Feb '23

14 Feb '23

Not completely sure if it's worth having for 8.0, but opening this to share the target I'm trying to reach. -- v11: ntdll: Add a heap thread affinity and per-affinity bin group cache. ntdll: Use atomics and lock-free list for bin groups. ntdll: Implement Low Fragmentation Heap frontend. ntdll: Split heap_resize_block into heap_resize_(block|large) helpers. ntdll: Count allocations and automatically enable LFH. ntdll: Increase heap block tail_size capacity to 16 bits. ntdll: Implement HeapCompatibilityInformation. ntdll: Fix HeapWalk with empty uncommitted consecutive subheaps. https://gitlab.winehq.org/wine/wine/-/merge_requests/1628

3 26

[PATCH 0/2] MR91: Two more fixes for functions.
by Zebediah Figura (＠zfigura) 14 Feb '23

14 Feb '23

-- https://gitlab.winehq.org/wine/vkd3d/-/merge_requests/91

5 5

[PATCH 0/1] MR92: vkd3d-shader/hlsl: Write SM4 break instructions.
by Giovanni Mascellani (＠giomasce) 14 Feb '23

14 Feb '23

-- https://gitlab.winehq.org/wine/vkd3d/-/merge_requests/92

3 3

Re: [PATCH v8 0/2] MR2175: oleaut32: fix VarAbs function for BSTR with positive values
by Jason Millard (＠jsm174) 14 Feb '23

14 Feb '23

On Tue Feb 14 16:20:32 2023 +0000, Jacek Caban wrote: > This will work, but while reviewing this I noticed that current handling > of VT_R4 and VT_R8 is not exactly right, overflow checks don't really > make sense. Fixing that allows to simplify VT_BSTR case a bit. Please > take a look at https://gitlab.winehq.org/jacek/wine/-/commits/varabs/ > and if it looks good to you, update MR with that branch (it also > contains BSTR allocation fixup). Thanks. Hopefully I rebased properly! -- https://gitlab.winehq.org/wine/wine/-/merge_requests/2175#note_24086

1 0

[PATCH 0/5] MR2176: uiautomationcore: UI Automation event raising function stubs.
by Connor McAdams (＠cmcadams) 14 Feb '23

14 Feb '23

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

3 6