[Bug 10327] New: Crash in WINPROC_CallProcAtoW

http://bugs.winehq.org/show_bug.cgi?id=10327

Summary: Crash in WINPROC_CallProcAtoW
           Product: Wine
           Version: CVS/GIT
          Platform: PC
        OS/Version: Linux
            Status: UNCONFIRMED
          Severity: major
          Priority: P2
         Component: wine-user
        AssignedTo: [e-mail]
        ReportedBy: [e-mail]

Created an attachment (id=8988)
 --> (http://bugs.winehq.org/attachment.cgi?id=8988)
sample backtrace

since I installed a new cvs build, a program which I use frequently (Action
Request System by Remedy) crashes in WINPROC_CallProcWtoA

I traced the stuff in the debugger and found that we have
in menu.c:1829: 
    menu->hWnd = CreateWindowExW( 0, (LPCWSTR)POPUPMENU_CLASS_ATOM, NULL,
     ....

So classname is an ATOMINT, and in WINPROC_CallProcWtoA it does the following:

class_lenW = strlenW(csW->lpszClass) * sizeof(WCHAR);
              RtlUnicodeToMultiByteSize(&class_lenA, csW->lpszClass,
class_lenW);
and later
              RtlUnicodeToMultiByteN(cls, class_lenA, NULL, csW->lpszClass,
class_lenW);
> From what I read in other functions, there should be something like

if (HIWORD(csA->lpszClass)) around it.

A sample backtrace is attached.

[Bug 10327] Crash in WINPROC_CallProcAtoW

http://bugs.winehq.org/show_bug.cgi?id=10327

Dan Kegel  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[e-mail]

--- Comment #1 from Dan Kegel   2007-11-06 10:05:29 ---
BTW, which version of Remedy is this?

Can you do a regression to find out which patch
introduced the failure?

[Bug 10327] Crash in WINPROC_CallProcAtoW

http://bugs.winehq.org/show_bug.cgi?id=10327

--- Comment #2 from Dmitry Timoshkov   2007-11-06 10:08:35 ---
The problem should be fixed in GIT now.

[Bug 10327] Crash in WINPROC_CallProcAtoW

http://bugs.winehq.org/show_bug.cgi?id=10327

--- Comment #3 from Michael Teske   2007-11-06 17:30:24 ---
it's 5.01.01. 
regression might take a while, but I'll try.

[Bug 10327] Crash in WINPROC_CallProcAtoW

http://bugs.winehq.org/show_bug.cgi?id=10327

--- Comment #4 from Michael Teske   2007-11-06 17:38:25 ---
Ok, I know that it worked on CVS from 2007-09-03. Do you need it more accurate?

[Bug 10327] Crash in WINPROC_CallProcAtoW

http://bugs.winehq.org/show_bug.cgi?id=10327

--- Comment #5 from James Hawkins   2007-11-06 17:40:13 ---
Michael, please read all the bug report comments, especially comment #2.  The
bug should be fixed, so please update to git and try again.

[Bug 10327] Crash in WINPROC_CallProcAtoW

http://bugs.winehq.org/show_bug.cgi?id=10327

--- Comment #6 from Michael Teske   2007-11-06 17:57:38 ---
sorry, somehow I ignored them since there were no notify mails. Anyway, I just
checked out from git (completely missed that) and started a build, will try
tomorrow.

[Bug 10327] Crash in WINPROC_CallProcAtoW

http://bugs.winehq.org/show_bug.cgi?id=10327

--- Comment #7 from Michael Teske   2007-11-07 02:29:55 ---
I just checked and it is fixed there. Thank you very much.

[Bug 10327] Crash in WINPROC_CallProcAtoW

http://bugs.winehq.org/show_bug.cgi?id=10327

Dmitry Timoshkov  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |RESOLVED
         Resolution|                            |FIXED

--- Comment #8 from Dmitry Timoshkov   2007-11-07 03:07:11 ---
Confirmed fixed.

[Bug 10327] Crash in WINPROC_CallProcAtoW

http://bugs.winehq.org/show_bug.cgi?id=10327

Dmitry Timoshkov  changed:

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

--- Comment #9 from Dmitry Timoshkov   2007-11-07 03:07:54 ---
Closing.

[Bug 10327] Crash in WINPROC_CallProcAtoW

http://bugs.winehq.org/show_bug.cgi?id=10327

Austin English  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
            Version|CVS/GIT                     |unspecified