[Bug 21820] New: 3d architecte from microapplication assert on startup

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

           Summary: 3d architecte from microapplication assert on startup
           Product: Wine
           Version: 1.1.39
          Platform: x86
               URL: http://www.microapp.com/logiciel_3d_architecte_expert_
                    cad_2010_2674.html
        OS/Version: Linux
            Status: UNCONFIRMED
          Severity: blocker
          Priority: P2
         Component: -unknown
        AssignedTo: wine-bugs(a)winehq.org
        ReportedBy: jeanluc.malet(a)gmail.com


see the attached snapshot for the error message

3d architecte from micro application (see url, use the evaluation version) show
this error message when starting and then exit

step to reproduce :
1) download and install the evaluation version
2) launch it

-- 
Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email
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.

[Bug 21820] 3d architecte from microapplication assert on startup

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





--- Comment #1 from jeanluc.malet(a)gmail.com  2010-02-22 09:40:03 ---
Created an attachment (id=26405)
 --> (http://bugs.winehq.org/attachment.cgi?id=26405)
the displayed error message

-- 
Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email
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.

[Bug 21820] 3d architecte from microapplication assert on startup

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


Austin English <austinenglish(a)gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Severity|blocker                     |normal




--- Comment #2 from Austin English <austinenglish(a)gmail.com>  2010-02-22 11:27:33 ---
Not a blocker. Please attach terminal output.

-- 
Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email
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.

[Bug 21820] 3d architecte from microapplication assert on startup

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


Andrew Nguyen <arethusa26(a)gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |download
             Status|UNCONFIRMED                 |NEW
                URL|http://www.microapp.com/log |http://download2.microapp.c
                   |iciel_3d_architecte_expert_ |om/telechargement/eval/2674
                   |cad_2010_2674.html          |_eval.exe
          Component|-unknown                    |user32
     Ever Confirmed|0                           |1




--- Comment #3 from Andrew Nguyen <arethusa26(a)gmail.com>  2010-02-22 21:52:29 ---
The only line that the application emits is:

fixme:reg:GetNativeSystemInfo (0x32f9e8) using GetSystemInfo()
> From +relay,+seh,+tid log:

001c:Call
user32.DialogBoxIndirectParamW(00000000,0050096c,00020022,00442a06,00500638)
ret=00442ccf
...
001c:Call window proc 0x7a50ac7c
(hwnd=0x10066,msg=WM_INITDIALOG,wp=00000000,lp=00500638)
001c:Call dialog proc 0x442a06
(hwnd=0x10066,msg=WM_INITDIALOG,wp=00000000,lp=00500638)
...
001c:Call KERNEL32.RaiseException(e06d7363,00000001,00000003,0032f0c0)
ret=0044ccd1
001c:trace:seh:raise_exception code=e06d7363 flags=1 addr=0x7b835e12
ip=7b835e12 tid=001c
...

For the WM_INITDIALOG message, MSDN states that the wParam value is:

"Handle to the control to receive the default keyboard focus. The system
assigns the default keyboard focus only if the dialog box procedure returns
TRUE."

In DIALOG_CreateIndirect in dlls/user32/dialog.c however:

    /* moved this from the top of the method to here as DIALOGINFO structure
    will be valid only after WM_CREATE message has been handled in DefDlgProc
    All the members of the structure get filled here using temp variables */
    dlgInfo = DIALOG_get_info( hwnd, TRUE );
    dlgInfo->hwndFocus   = 0;
...
    if (DIALOG_CreateControls32( hwnd, dlgTemplate, &template, hInst, unicode
))
    {
        /* Send initialisation messages and set focus */

        if (dlgProc)
        {
            if (SendMessageW( hwnd, WM_INITDIALOG, (WPARAM)dlgInfo->hwndFocus,
param ) &&
                ((~template.style & DS_CONTROL) || (template.style &
WS_VISIBLE)))
...

it seems that dlgInfo->hwndFocus is never initialized to an appropriate handle
value, causing the application to assert. Initializing dlgInfo->hwndFocus makes
the assertion failure go away, although the application simply terminates
silently after doing this.

-- 
Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email
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.

[Bug 21820] 3d architecte from microapplication assert on startup

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





--- Comment #4 from Andrew Nguyen <arethusa26(a)gmail.com>  2010-02-22 22:09:11 ---
Created an attachment (id=26420)
 --> (http://bugs.winehq.org/attachment.cgi?id=26420)
user32: Initialize the focus handle before sending WM_INITDIALOG

This seems to allow the dialog to be displayed correctly, although the app runs
into more problems later.

-- 
Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email
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.

[Bug 21820] 3d architecte from microapplication assert on startup

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





--- Comment #5 from Andrew Nguyen <arethusa26(a)gmail.com>  2010-02-23 01:25:30 ---
I sent a patch:

http://source.winehq.org/patches/data/58688

-- 
Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email
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.

[Bug 21820] 3d architecte from microapplication assert on startup

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


Andrew Nguyen <arethusa26(a)gmail.com> changed:

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




--- Comment #6 from Andrew Nguyen <arethusa26(a)gmail.com>  2010-02-23 11:39:46 ---
This is fixed by commit 0b2301286796df64959d910b4afb3609b6da3e41. Please open
new bugs for other problems with the application.

-- 
Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email
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.

[Bug 21820] 3d architecte from microapplication assert on startup

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


Alexandre Julliard <julliard(a)winehq.org> changed:

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




--- Comment #7 from Alexandre Julliard <julliard(a)winehq.org>  2010-03-05 12:43:32 ---
Closing bugs fixed in 1.1.40.

-- 
Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email
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.