November 2024 - wine-gitlab

[PATCH v3 0/2] MR6783: winegstreamer: Fix a memory/resource leak.
by Conor McCarthy (＠cmccarthy) 08 Nov '24

08 Nov '24

References held by contained stream objects must use an internal refcount, otherwise they prevent release of the media source if Start() and Shutdown() are not called. -- v3: winegstreamer: Introduce an internal refcount in media source objects. mfplat/tests: Test the refcount before the media source is released. https://gitlab.winehq.org/wine/wine/-/merge_requests/6783

3 4

[PATCH v3 0/1] MR6790: ntdll: use __wine_unix_spawnvp() to invoke unmount command.
by Michael Lelli (＠toadking) 08 Nov '24

08 Nov '24

Fix umounting filesystems mounted on paths with spaces. Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=57391 -- v3: ntdll: use __wine_unix_spawnvp() to invoke unmount command. https://gitlab.winehq.org/wine/wine/-/merge_requests/6790

2 2

[PATCH v2 0/1] MR6790: ntdll: use posix_spawn() to invoke unmount command instead of system().
by Michael Lelli (＠toadking) 08 Nov '24

08 Nov '24

Fix umounting filesystems mounted on paths with spaces. Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=57391 -- v2: ntdll: use posix_spawn() to invoke unmount command instead of system(). https://gitlab.winehq.org/wine/wine/-/merge_requests/6790

3 2

[PATCH 0/1] MR6790: ntdll: use fork()/exec() to invoke unmount command instead of system().
by Michael Lelli (＠toadking) 08 Nov '24

08 Nov '24

Fix umounting filesystems mounted on paths with spaces. Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=57391 -- https://gitlab.winehq.org/wine/wine/-/merge_requests/6790

3 2

[PATCH 0/1] MR6772: d3dx9: Add semi-stub for D3DXOptimizeVertices.
by Alex Henrie (＠alexhenrie) 08 Nov '24

08 Nov '24

This submission is practically identical to [Vijay Kiran Kamuju's improved version](https://list.winehq.org/mailman3/hyperkitty/list/wine-devel@winehq… of [Christian Costa's D3DXOptimizeVertices patch in Staging](https://github.com/wine-staging/wine-staging/blob/master/patches/d…. The only changes I made were adding the "Vertex order will not be optimal" message to match [the similar message in D3DXOptimizeFaces](https://gitlab.winehq.org/wine/wine/-/blob/eeabbe812ffc3…, removing the extra newlines around the function, and changing `%x` to `%lx` in the tests to resolve warnings. Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=47776 Signed-off-by: Vijay Kiran Kamuju <infyquest(a)gmail.com> Signed-off-by: Alex Henrie <alexhenrie24(a)gmail.com> -- https://gitlab.winehq.org/wine/wine/-/merge_requests/6772

3 5

Re: [PATCH v7 0/2] MR6737: comctl32/toolbar: forward unhandled WM_NOTIFY
by Zhiyi Zhang (＠zhiyi) 08 Nov '24

08 Nov '24

On Thu Nov 7 19:56:04 2024 +0000, Alanas wrote: > <div><a target="_blank" > href="https://gitlab.winehq.org/wine/wine/-/blob/6db2812411d45ca4b4d15dea9a00577e…"><code>pager.c</code>:<pre><code > lang="c">static BOOL PAGER_AdjustBuffer(PAGER_INFO *infoPtr, INT > size)
{
 if (!infoPtr->pwszBuffer)
 > infoPtr->pwszBuffer = Alloc(size);
 else if > (infoPtr->nBufferSize < size)
 infoPtr->pwszBuffer = > ReAlloc(infoPtr->pwszBuffer, size);

 if > (!infoPtr->pwszBuffer) return FALSE;
 if (infoPtr->nBufferSize > < size) infoPtr->nBufferSize = size;

 return > TRUE;
}</code></pre></a>Why is this function designed to leak old > <code>infoPtr->pwszBuffer</code> when <code>ReAlloc(infoPtr->pwszBuffer, > size)</code> returns <code>NULL</code>?</div> No. It's a bug. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/6737#note_86878

1 0

Re: [PATCH v7 0/3] MR2377: sppc: Add stubs for MS Office.
by Daniel Tang (＠danielzgtg) 07 Nov '24

07 Nov '24

Closing per merge unlikelihood. Alternatives approaches are at https://gitlab.winehq.org/wine/wine/-/merge_requests/5522#note_86850 . -- https://gitlab.winehq.org/wine/wine/-/merge_requests/2377#note_86876

1 0

Re: [PATCH v7 0/3] MR2377: sppc: Add stubs for MS Office. - closed
by Daniel Tang (＠danielzgtg) 07 Nov '24

07 Nov '24

This merge request was closed by Daniel Tang. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/2377

1 0

Re: [PATCH v4 0/1] MR2378: kernelbase: Trick V8's CHECK(old_protection == PAGE_READWRITE).
by Paul Gofman (＠gofman) 07 Nov '24

07 Nov '24

On Thu Nov 7 23:04:24 2024 +0000, Elizabeth Figura wrote: > > > Proton hack won't be acceptable upstream > > > > I learned that Wine and wine-staging doesn't like app-specific code. > However, Proton has [a lot of app-specific > workarounds](https://github.com/ValveSoftware/Proton/blob/1a73b04e6cdf29297…. > Since Microsoft > [famously](https://arstechnica.com/gadgets/2022/10/windows-95-went-the-extra… > has app-specific workarounds (which Wine needs to emulate for full > conformance), our own workaround for Microsoft Office wouldn't be much > different on the technical side. On the social side, Wine wouldn't > accept stuff like that, but Proton has a history of this. > Proton already has a hack for this issue. That hack is game specific, not enabled by default (as currently considered too dangerous and not that often needed). It is not enabled by MS Office of course. Maybe it can be, provided it works with Proton with the hack. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/2378#note_86875

1 0

Re: [PATCH v4 0/1] MR2378: kernelbase: Trick V8's CHECK(old_protection == PAGE_READWRITE).
by Brendan Shanks (＠bshanks) 07 Nov '24

07 Nov '24

Also, I landed a fix for [this issue](https://bugs.winehq.org/show_bug.cgi?id=53835) in upstream Chromium 110/V8 11.0 (and it was backported to later releases of 109). There is software (various game launchers) still using affected versions (106-109), but I'd hope that Office would be more up-to-date. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/2378#note_86874

1 0

2025

2024

2023

2022

wine-gitlab November 2024