[Bug 52322] New: wineserver memory leak during HTTPs download (Genshin Impact Launcher, Qt5)

List overview All Threads

newer

older

[Bug 35254] New: Freemake Video...

[Bug 54819] New: DnsQuery_A()...

WineHQ Bugzilla

2 Jan 2022 2 Jan '22

11:14 a.m.

https://bugs.winehq.org/show_bug.cgi?id=52322

Bug ID: 52322 Summary: wineserver memory leak during HTTPs download (Genshin Impact Launcher, Qt5) Product: Wine Version: 7.0-rc2 Hardware: x86-64 OS: Linux Status: UNCONFIRMED Severity: normal Priority: P2 Component: -unknown Assignee: wine-bugs@winehq.org Reporter: mk939@ymail.com Distribution: ---

The Genshin Impact launcher application downloads *.zip archives from an HTTPs server using the Qt5 engine. During the download, the memory footprint of wineserver increases proportionally (approx 1:2) to the amount of downloaded data. This results in systems running out of free RAM due to the 30+ GB download size (!)

This issue is not due to a recent change in Wine. Another user already reported this to me 7 months ago (Wine 6.8-1).

Reproduction report 1: (4 months ago)

* Wine 6.13 (vanilla) * Launcher version: 2.11.2.0 (Qt5 version: 5.12.5.0)

Reproduction report 2: (new)

* Wine 7.0-rc2 (staging), Wine 7.0-rc2 (Lutris) * Launcher version: 2.14.1.0 (Qt5 version: 5.12.5.0)

Launcher download URL for reproduction:

https://genshinimpact.mihoyo.com/client_app/download/launcher/20211221171228...

Due to DNS routing issues I yet cannot reproduce this bug with a recent vanilla Wine version, but will give it a try as soon it is sorted out.

-- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.

Show replies by date

WineHQ Bugzilla

2 Jan 2 Jan

10:10 p.m.

New subject: [Bug 52322] wineserver memory leak during HTTPs download (Genshin Impact Launcher, Qt5)

https://bugs.winehq.org/show_bug.cgi?id=52322

Fabian Maurer dark.shadow4@web.de changed:

--- Comment #1 from Fabian Maurer dark.shadow4@web.de --- Confirming using wine-7.0-rc3. Tried using valgrind, but according to that, nothing got leaked...

-- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.

WineHQ Bugzilla

10:10 p.m.

New subject: [Bug 52322] wineserver memory leak during HTTPs download (Genshin Impact Launcher, Qt5)

https://bugs.winehq.org/show_bug.cgi?id=52322

Fabian Maurer dark.shadow4@web.de changed:

--- Comment #2 from Fabian Maurer dark.shadow4@web.de --- Forgot to confirm.

-- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.

WineHQ Bugzilla

9 Jan 9 Jan

10:01 p.m.

New subject: [Bug 52322] wineserver memory leak during HTTPs download (Genshin Impact Launcher, Qt5)

https://bugs.winehq.org/show_bug.cgi?id=52322

Bernhard Übelacker bernhardu@mailbox.org changed:

What |Removed |Added ---------------------------------------------------------------------------- CC| |bernhardu@mailbox.org

--- Comment #3 from Bernhard Übelacker bernhardu@mailbox.org --- Created attachment 71583 --> https://bugs.winehq.org/attachment.cgi?id=71583 test-GetVolumeInformationByHandleW.cpp

I tried if I can reproduce this issue and found this while downloading:

- wineserver memory usage increases slightly over time.

- One winedevice.exe memory usage increases a lot more.

- taskmgr, with column handle count shows a increasing number. (For some reason taskmgr stuck at 2,097,152 and increased no more.)

- When launcher.exe is closed the number of handles and memory stayed at high values.

It looks like one thread of launcher.exe is doing a call to GetVolumeInformationW in a tight loop.

Attached test program demonstrates this and shows the same effects.

-- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.

WineHQ Bugzilla

11:05 p.m.

New subject: [Bug 52322] wineserver memory leak during HTTPs download (Genshin Impact Launcher, Qt5)

https://bugs.winehq.org/show_bug.cgi?id=52322

Anastasius Focht focht@gmx.net changed:

--- Comment #4 from Anastasius Focht focht@gmx.net --- Hello folks,

I've created a snapshot via Internet Archive here:

https://web.archive.org/web/20220109222357/https://genshinimpact.mihoyo.com/...

https://www.virustotal.com/gui/file/36849a4fbae38d5493d2636f35c14592e73bbd12...

$ sha1sum GenshinImpact_install_mihoyo_20211214142226.exe 524c826742a6fdb1915ab6a26352e38e42fd074f GenshinImpact_install_mihoyo_20211214142226.exe

$ du -sh GenshinImpact_install_mihoyo_20211214142226.exe 119M GenshinImpact_install_mihoyo_20211214142226.exe

Regards

-- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.

WineHQ Bugzilla

11 Jan 11 Jan

9:26 p.m.

New subject: [Bug 52322] wineserver memory leak during HTTPs download (Genshin Impact Launcher, Qt5)

https://bugs.winehq.org/show_bug.cgi?id=52322

--- Comment #5 from Bernhard Übelacker bernhardu@mailbox.org --- Created attachment 71601 --> https://bugs.winehq.org/attachment.cgi?id=71601 ntoskrnl: Free memory before pointer gets set to NULL.

Freeing memory in winedevice.exe: For some reason in dispatch_volume context->in_buff is set to NULL. By that the memory seems to get never freed. This patch attempts to release the memory before the pointer is set to NULL.

But I am not sure if this could be the right place or the right way to do this.

-- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.

WineHQ Bugzilla

9:29 p.m.

New subject: [Bug 52322] wineserver memory leak during HTTPs download (Genshin Impact Launcher, Qt5)

https://bugs.winehq.org/show_bug.cgi?id=52322

--- Comment #6 from Bernhard Übelacker bernhardu@mailbox.org --- Created attachment 71602 --> https://bugs.winehq.org/attachment.cgi?id=71602 server: Close handle of the previous device close request.

Releasing the handle used for the IRP_MJ_CLOSE roundtrip. Otherwise the handle is not used anymore as far as I see, but winedevice.exe gets accounted for it.

But the same here, I am not sure if this is the expected way or place to release that handle.

At least with both patches applied, memory and handle counter stays low while running the first attached test program.

-- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.

WineHQ Bugzilla

16 Mar 16 Mar

9:45 a.m.

New subject: [Bug 52322] wineserver memory leak during HTTPs download (Genshin Impact Launcher, Qt5)

https://bugs.winehq.org/show_bug.cgi?id=52322

Bernhard Übelacker bernhardu@mailbox.org changed:

--- Comment #7 from Bernhard Übelacker bernhardu@mailbox.org --- Comment on attachment 71602 --> https://bugs.winehq.org/attachment.cgi?id=71602 server: Close handle of the previous device close request.

A similar patch got committed making attachement 71602 obsolete: https://source.winehq.org/git/wine.git/commitdiff/2fc3d7eb2912b189ab38c0683f...

The attachment 71601 still helps to avoid the increased memory usage of winedevice.exe when running the test program in attachement 71583.

-- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.

WineHQ Bugzilla

19 Mar 19 Mar

2:50 p.m.

New subject: [Bug 52322] wineserver memory leak during HTTPs download (Genshin Impact Launcher, Qt5)

https://bugs.winehq.org/show_bug.cgi?id=52322

--- Comment #8 from Bernhard Übelacker bernhardu@mailbox.org --- Submitted attachment 71601 for review: https://source.winehq.org/patches/data/230017

-- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.

WineHQ Bugzilla

2 Apr 2 Apr

7:31 a.m.

New subject: [Bug 52322] wineserver memory leak during HTTPs download (Genshin Impact Launcher, Qt5)

https://bugs.winehq.org/show_bug.cgi?id=52322

ppwwyyxxc@gmail.com changed:

What |Removed |Added ---------------------------------------------------------------------------- CC| |ppwwyyxxc@gmail.com

-- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.

WineHQ Bugzilla

4 May 4 May

6:17 p.m.

New subject: [Bug 52322] wineserver memory leak during HTTPs download (Genshin Impact Launcher, Qt5)

https://bugs.winehq.org/show_bug.cgi?id=52322

--- Comment #9 from mk939@ymail.com --- (In reply to Bernhard Übelacker from comment #8)

...

Submitted attachment 71601 [details] for review: https://source.winehq.org/patches/data/230017

Thank you for the patch. Is there yet any progress in this regard?

-- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.

WineHQ Bugzilla

2 Jun 2 Jun

8:33 p.m.

New subject: [Bug 52322] wineserver memory leak during HTTPs download (Genshin Impact Launcher, Qt5)

https://bugs.winehq.org/show_bug.cgi?id=52322

mashkal2000@gmail.com changed:

What |Removed |Added ---------------------------------------------------------------------------- CC| |mashkal2000@gmail.com

-- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.

WineHQ Bugzilla

2 Nov 2 Nov

5:49 a.m.

New subject: [Bug 52322] wineserver memory leak during HTTPs download (Genshin Impact Launcher, Qt5)

https://bugs.winehq.org/show_bug.cgi?id=52322

--- Comment #10 from Fabian Maurer dark.shadow4@web.de --- Any news on this? The leak is still present with wine-7.20 and the newest Genshin launcher.

-- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.

WineHQ Bugzilla

19 Mar 19 Mar

4:55 p.m.

New subject: [Bug 52322] wineserver memory leak during HTTPs download (Genshin Impact Launcher, Qt5)

https://bugs.winehq.org/show_bug.cgi?id=52322

--- Comment #11 from Bernhard Übelacker bernhardu@mailbox.org --- Sorry for the delay.

The test application test-GetVolumeInformationByHandleW.cpp still triggers the growing memory usage of winedevice.exe in wine-8.4.

I created this unmodified merge request: https://gitlab.winehq.org/wine/wine/-/merge_requests/2439

-- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.

WineHQ Bugzilla

19 Apr 19 Apr

5:52 p.m.

New subject: [Bug 52322] wineserver memory leak during HTTPs download (Genshin Impact Launcher, Qt5)

https://bugs.winehq.org/show_bug.cgi?id=52322

--- Comment #12 from Bernhard Übelacker bernhardu@mailbox.org --- This bug should now be completely fixed with: https://gitlab.winehq.org/wine/wine/-/commit/07c0cd6ba5f4b1d6ef738ad7f3cad99... (I cannot change this bug's state.)

In the MR is another application "trackmania leaking memory on proton" mentioned - this seems discussed here: https://github.com/ValveSoftware/Proton/issues/6507#issuecomment-1493299894 I found no separate wine bug, but it might be fixed by this commit too.

-- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.

WineHQ Bugzilla

6:03 p.m.

New subject: [Bug 52322] wineserver memory leak during HTTPs download (Genshin Impact Launcher, Qt5)

https://bugs.winehq.org/show_bug.cgi?id=52322

Zeb Figura z.figura12@gmail.com changed:

--- Comment #13 from Zeb Figura z.figura12@gmail.com --- (In reply to Bernhard Übelacker from comment #12)

...

This bug should now be completely fixed with: https://gitlab.winehq.org/wine/wine/-/commit/ 07c0cd6ba5f4b1d6ef738ad7f3cad9992947dd3d

Marking fixed, thank you!

-- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.

WineHQ Bugzilla

28 Apr 28 Apr

8:07 p.m.

New subject: [Bug 52322] wineserver memory leak during HTTPs download (Genshin Impact Launcher, Qt5)

https://bugs.winehq.org/show_bug.cgi?id=52322

Alexandre Julliard julliard@winehq.org changed:

What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |CLOSED

--- Comment #14 from Alexandre Julliard julliard@winehq.org --- Closing bugs fixed in 8.7.

-- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.

WineHQ Bugzilla

29 May 29 May

9:03 p.m.

New subject: [Bug 52322] wineserver memory leak during HTTPs download (Genshin Impact Launcher, Qt5)

https://bugs.winehq.org/show_bug.cgi?id=52322

Michael Stefaniuc mstefani@winehq.org changed:

What |Removed |Added ---------------------------------------------------------------------------- Target Milestone|--- |8.0.x

-- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.

WineHQ Bugzilla

19 Jul 19 Jul

5:31 p.m.

New subject: [Bug 52322] wineserver memory leak during HTTPs download (Genshin Impact Launcher, Qt5)

https://bugs.winehq.org/show_bug.cgi?id=52322

Michael Stefaniuc mstefani@winehq.org changed:

What |Removed |Added ---------------------------------------------------------------------------- Target Milestone|8.0.x |---

--- Comment #15 from Michael Stefaniuc mstefani@winehq.org --- Removing the 8.0.x milestone from bug fixes included in 8.0.2.

-- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.

542

Age (days ago)

1105

Last active (days ago)

wine-bugs@winehq.org

18 comments

1 participants

tags (0)

participants (1)

WineHQ Bugzilla