[Bug 38021] New: ntdll:load_dll doesn't check for correct dll ARCH

List overview All Threads
Download

newer

older

[Bug 36349] New: valgrind shows a...

[Bug 36267] New: valgrind shows an...

wine-bugs＠winehq.org

6 Feb 2015 6 Feb '15

12:18 p.m.

https://bugs.winehq.org/show_bug.cgi?id=38021 Bug ID: 38021 Summary: ntdll:load_dll doesn't check for correct dll ARCH Product: Wine Version: unspecified Hardware: x86-64 OS: Linux Status: UNCONFIRMED Severity: normal Priority: P2 Component: ntdll Assignee: wine-bugs(a)winehq.org Reporter: patrick1804(a)web.de Distribution: --- Problem 1: ntdll:load_dll doesn't check if the found DLL has the wrong arch. For example: 64bit application want's to load xinput1_3.dll. There's a 32bit xinput1_3.dll in the same folder. Expected behavior: It should ignore this dll, instead of trying to open it. Problem 2: ntdll:load_dll doesn't check other locations for this DLL once a matching one has been found, even when it has the wrong arch and couldn't be loaded. For example: 64bit application want's to load xinput1_3.dll. There's a 32bit xinput1_3.dll in the same folder. It tries to open the file and fails. It doesn't check other folders like System32. The application doesn't start. Expected behavior: load_dll looks for 64bit xinput1_3.dll in system32 and starts. -- 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

wine-bugs＠winehq.org

6 Feb 6 Feb

4:24 p.m.

New subject: [Bug 38021] ntdll:load_dll doesn't check for correct dll ARCH

https://bugs.winehq.org/show_bug.cgi?id=38021 --- Comment #1 from Patrick Rudolph <patrick1804(a)web.de> --- Created attachment 50663 --> https://bugs.winehq.org/attachment.cgi?id=50663 load_dll: iterate over all folders in path, POC this patch allows load_dll to iterate over every folder in the given path and check for a matching dll until one could be loaded. As soon as a dll is found and could be loaded it returns, introducing zero overhead for the most applications. Only on WoW it might try multiple dlls until a matching one is found. With this patch Planetside 2 x64 doesn't complain about missing xinput1_3 any more. -- 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.

wine-bugs＠winehq.org

7 Feb 7 Feb

3:08 p.m.

New subject: [Bug 38021] ntdll:load_dll doesn't check for correct dll ARCH

https://bugs.winehq.org/show_bug.cgi?id=38021 Patrick Rudolph <patrick1804(a)web.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #50663|0 |1 is obsolete| | --- Comment #2 from Patrick Rudolph <patrick1804(a)web.de> --- Created attachment 50666 --> https://bugs.winehq.org/attachment.cgi?id=50666 iterate over all folders in path, POC #2 this patch fixes a problem on 32bit with empty path and fixes a warning. With this patch Planetside2 x64 is able to find xinput1_3.dll in system32. -- 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.

wine-bugs＠winehq.org

11 Jul 11 Jul

9:28 a.m.

New subject: [Bug 38021] ntdll:load_dll doesn't check for correct dll ARCH

https://bugs.winehq.org/show_bug.cgi?id=38021 Linards <linards.liepins(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |linards.liepins(a)gmail.com --- Comment #3 from Linards <linards.liepins(a)gmail.com> --- (In reply to Patrick Rudolph from comment #2)

...

Created attachment 50666 [details] iterate over all folders in path, POC #2 this patch fixes a problem on 32bit with empty path and fixes a warning. With this patch Planetside2 x64 is able to find xinput1_3.dll in system32.

Hi. Could you please provide the output of 'wine --version'? -- 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.

wine-bugs＠winehq.org

13 Jul 13 Jul

4:40 p.m.

New subject: [Bug 38021] ntdll:load_dll doesn't check for correct dll ARCH

https://bugs.winehq.org/show_bug.cgi?id=38021 Ken Sharp <imwellcushtymelike(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Keywords| |download, patch, source --- Comment #4 from Ken Sharp <imwellcushtymelike(a)gmail.com> --- Thanks for the patches. Patches aren't picked up from Bugzilla. If your patch is ready you can submit it via e-mail: http://wiki.winehq.org/SubmittingPatches -- 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.

wine-bugs＠winehq.org

18 Jul 18 Jul

11:14 p.m.

New subject: [Bug 38021] ntdll:load_dll doesn't check for correct dll ARCH

https://bugs.winehq.org/show_bug.cgi?id=38021 Sebastian Lackner <sebastian(a)fds-team.de> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |sebastian(a)fds-team.de --- Comment #5 from Sebastian Lackner <sebastian(a)fds-team.de> --- The patch proposed in comment #2 is wrong, Michaels attempt to fix this issue is available here: https://github.com/wine-compholio/wine-staging/blob/master/patches/ntdll-Lo… I have reviewed the patch and it should be correct, however I don't have any of the affected apps, so some testers would be very appreciated. ;) -- 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.

wine-bugs＠winehq.org

11:15 p.m.

New subject: [Bug 38021] ntdll:load_dll doesn't check for correct dll ARCH

https://bugs.winehq.org/show_bug.cgi?id=38021 Sebastian Lackner <sebastian(a)fds-team.de> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |michael(a)fds-team.de -- 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.

wine-bugs＠winehq.org

19 Jul 19 Jul

9:48 a.m.

New subject: [Bug 38021] 64-bit Planetside 2 fails to load bundled xinput1_3.dll (loader must check architecture before trying to load module)

wine-bugs＠winehq.org

28 Sep 28 Sep

11:59 p.m.

New subject: [Bug 38021] 64-bit Planetside 2 fails to load bundled xinput1_3.dll (loader must check architecture before trying to load module)

https://bugs.winehq.org/show_bug.cgi?id=38021 Michael Müller <michael(a)fds-team.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |STAGED Staged patchset| |https://github.com/wine-com | |pholio/wine-staging/tree/ma | |ster/patches/ntdll-Loader_M | |achine_Type -- 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.

wine-bugs＠winehq.org

20 Oct 20 Oct

1:18 p.m.

New subject: [Bug 38021] 64-bit Planetside 2 fails to load bundled xinput1_3.dll (loader must check architecture before trying to load module)

https://bugs.winehq.org/show_bug.cgi?id=38021 Justas L <bucaneer(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |bucaneer(a)gmail.com --- Comment #7 from Justas L <bucaneer(a)gmail.com> --- (In reply to Sebastian Lackner from comment #5)

...

The patch proposed in comment #2 is wrong, Michaels attempt to fix this issue is available here: https://github.com/wine-compholio/wine-staging/blob/master/patches/ntdll- Loader_Machine_Type/0001-ntdll-Check-architecture-before-loading-module.patch I have reviewed the patch and it should be correct, however I don't have any of the affected apps, so some testers would be very appreciated. ;)

The staged patch (either applied individually, or as part of wine-staging) fixes this bug. -- 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.

wine-bugs＠winehq.org

24 Nov 24 Nov

12:21 p.m.

New subject: [Bug 38021] 64-bit Planetside 2 fails to load bundled xinput1_3.dll (loader must check architecture before trying to load module)

https://bugs.winehq.org/show_bug.cgi?id=38021 --- Comment #8 from Justas L <bucaneer(a)gmail.com> --- Could this patch make it in for the 1.8 release? The other blocker bug for Planetside 2 (bug 27680) should be resolved next week if GCC 5.3 gets released as planned, so if this patch is committed the game will be playable with vanilla Wine once again. -- 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.

wine-bugs＠winehq.org

25 Nov 25 Nov

8:29 p.m.

New subject: [Bug 38021] 64-bit Planetside 2 fails to load bundled xinput1_3.dll (loader must check architecture before trying to load module)

https://bugs.winehq.org/show_bug.cgi?id=38021 --- Comment #9 from Sebastian Lackner <sebastian(a)fds-team.de> --- (In reply to Justas L from comment #8)

...

Could this patch make it in for the 1.8 release? The other blocker bug for Planetside 2 (bug 27680) should be resolved next week if GCC 5.3 gets released as planned, so if this patch is committed the game will be playable with vanilla Wine once again.

We are during the code freeze at the moment, which means only patches with a small risk of regressions are accepted. Although this specific patch was tested in Wine Staging for some time, I would still consider it a bit too critical - changes in the Wine loader affect all applications, and its impossible to test everything in advance. :/ I hope you are aware that you can build a custom Wine version with this patch (or just use Wine Staging) in the meantime, so no need to wait when you want to play the game. -- 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.

wine-bugs＠winehq.org

15 Mar 15 Mar

12:37 a.m.

New subject: [Bug 38021] 64-bit Planetside 2 fails to load bundled xinput1_3.dll (loader must check architecture before trying to load module)

https://bugs.winehq.org/show_bug.cgi?id=38021 Andrew Chin <achin+wine(a)eminence32.net> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |achin+wine(a)eminence32.net --- Comment #10 from Andrew Chin <achin+wine(a)eminence32.net> --- I also confirm that 0001-ntdll-Check-architecture-before-loading-module.patch fixes this problem (when applied on top of wine-1.9.5) -- 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.

wine-bugs＠winehq.org

1 Jun 1 Jun

4:46 p.m.