[Bug 48480] New: dlls/krnl386.exe16 - thunk.c argument logic incorrect for CallProcEx32W16 - wine-bugs

List overview All Threads

newer

[Bug 48480] New: dlls/krnl386.exe16 - thunk.c argument logic incorrect for CallProcEx32W16

older

[Bug 48371] New: Home Designer...

[Bug 48561] New: Cadence Allegro...

WineHQ Bugzilla

18 Jan 2020 18 Jan '20

5:28 p.m.

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

Bug ID: 48480 Summary: dlls/krnl386.exe16 - thunk.c argument logic incorrect for CallProcEx32W16 Product: Wine Version: 5.0-rc6 Hardware: x86 OS: Mac OS X Status: UNCONFIRMED Severity: normal Priority: P2 Component: -unknown Assignee: wine-bugs@winehq.org Reporter: dirk.niggemann@gmail.com

CallProcEx32W16 takes a 'nrofargs' argument which may have the hight bit set by the caller (CPEX_DEST_CDECL). Internally the function processes nrofargs in a for loop but does not clear the high bit before looping.

This line (2475):

for (i=0;i<nrofargs;i++)

This needs to be changed to support correct argument counting to:

for (i=0;i<(nrofargs & ~CPEX_DEST_CDECL);i++)

Otherwise this routine crashes when called with CPEX_DEST_CDECL set.

-- 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

18 Jan 18 Jan

5:36 p.m.

New subject: [Bug 48480] dlls/krnl386.exe16 - thunk.c argument logic incorrect for CallProcEx32W16

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

--- Comment #1 from Alexandre Julliard julliard@winehq.org --- Do you have an application that uses 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

5:52 p.m.

New subject: [Bug 48480] dlls/krnl386.exe16 - thunk.c argument logic incorrect for CallProcEx32W16

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

--- Comment #2 from Dirk Niggemann dirk.niggemann@gmail.com ---

Yes, trying to run Hp/Agilent Chemstation G1701BA MSCONFIG.EXE (configuration program).

This is a now-ancient program to control GC/MS systems via GPIB (released 2000). Can't use any later versions as HP discontinued support for GPIB-based hardware after this version.

Was planning on building a HP SICL to linux GPIB dll at some point to make this program fully work under wine.

As it is the packages install OK and the data analysis program works (ish) but you have to copy the config (still uses win.ini!) from a working systems.

As it is MSCONFIG.EXE crashes during startup with an unhandled exception.

-- 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:04 p.m.

New subject: [Bug 48480] dlls/krnl386.exe16 - thunk.c argument logic incorrect for CallProcEx32W16

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

--- Comment #3 from Dirk Niggemann dirk.niggemann@gmail.com ---

Yes, trying to run Hp/Agilent Chemstation G1701BA MSCONFIG.EXE (configuration program).

This is a now-ancient program to control GC/MS systems via GPIB (released 2000). Can't use any later versions as HP discontinued support for GPIB-based hardware after this version.

Was planning on building a HP SICL to linux GPIB dll at some point to make this program fully work under wine.

As it is the packages install OK and the data analysis program works (ish) but you have to copy the config (still uses win.ini!) from a working systems.

As it is MSCONFIG.EXE crashes during startup with an unhandled exception.

-- 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 Feb 6 Feb

7:09 a.m.

New subject: [Bug 48480] dlls/krnl386.exe16 - thunk.c argument logic incorrect for CallProcEx32W16

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

Nikolay Sivov bunglehead@gmail.com changed:

--- Comment #4 from Nikolay Sivov bunglehead@gmail.com --- Hi, Dirk.

Thanks for the patch. Could we mark this report fixed?

-- 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

7 Feb 7 Feb

4:46 a.m.

New subject: [Bug 48480] dlls/krnl386.exe16 - thunk.c argument logic incorrect for CallProcEx32W16

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

Alistair Leslie-Hughes leslie_alistair@hotmail.com changed:

--- Comment #5 from Alistair Leslie-Hughes leslie_alistair@hotmail.com --- Fixed by https://source.winehq.org/git/wine.git/?a=commit;h=c57483ba4d77d04a7af120a6e...

-- 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 Feb 16 Feb

9:25 p.m.

New subject: [Bug 48480] dlls/krnl386.exe16 - thunk.c argument logic incorrect for CallProcEx32W16

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

Alexandre Julliard julliard@winehq.org changed:

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

--- Comment #6 from Alexandre Julliard julliard@winehq.org --- Closing bugs fixed in 5.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.

WineHQ Bugzilla

2 May 2 May

1:42 p.m.

New subject: [Bug 48480] dlls/krnl386.exe16 - thunk.c argument logic incorrect for CallProcEx32W16

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

Michael Stefaniuc mstefani@winehq.org changed:

What |Removed |Added ---------------------------------------------------------------------------- Target Milestone|--- |5.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

2 Jun 2 Jun

10:14 p.m.

New subject: [Bug 48480] dlls/krnl386.exe16 - thunk.c argument logic incorrect for CallProcEx32W16

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

Michael Stefaniuc mstefani@winehq.org changed:

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

--- Comment #7 from Michael Stefaniuc mstefani@winehq.org --- Removing the 5.0.x milestone from bug fixes included in 5.0.1.

-- 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.

1682

Age (days ago)

1818

Last active (days ago)

wine-bugs@winehq.org

8 comments

1 participants

tags (0)

participants (1)

WineHQ Bugzilla