[Bug 16498] New: hex workshop crashes in msi

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

Summary: hex workshop crashes in msi
           Product: Wine
           Version: 1.1.10
          Platform: Other
               URL: http://www.bpsoft.com/
        OS/Version: other
            Status: NEW
          Severity: enhancement
          Priority: P2
         Component: -unknown
        AssignedTo: [e-mail]
        ReportedBy: [e-mail]

a user reported this crash in appdb

[Bug 16498] hex workshop crashes in msi

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

--- Comment #1 from Louis Lenders   2008-12-14 04:04:35 ---
Created an attachment (id=17914)
 --> (http://bugs.winehq.org/attachment.cgi?id=17914)
crash log

[Bug 16498] hex workshop crashes in msi

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

Louis Lenders  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
          Component|-unknown                    |msi
           Keywords|                            |download, Installer

[Bug 16498] hex workshop crashes in msi

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

ddcc  changed:

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

[Bug 16498] hex workshop crashes in msi

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

GameZelda  changed:

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

--- Comment #2 from GameZelda   2008-12-21 04:38:46 ---
The process crashes on "strlenW" (unicode.h), since a NULL pointer is passed
from "resolve_keypath" (called on "comp->FullKeypath = resolve_keypath(
package, comp );")

The NULL pointer comes from the following line in resolve_keypath:
key = MSI_RecordGetString(row, 3);

The record type is "MSIFIELD_NULL", but it should be "MSIFIELD_WSTR".
"MSI_RecordGetString" returns NULL since of that, and this NULL pointer is
passed to strlenW.

[Bug 16498] hex workshop crashes in msi

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

--- Comment #3 from Luca Bennati   2009-08-12 10:23:22 ---
Created an attachment (id=23022)
 --> (http://bugs.winehq.org/attachment.cgi?id=23022)
log of the setup crash

Still valid in wine-1.1.27-63-g862bd3c e Hex Workshop 5.14
Attached is the log of the setup process till it crashes with a messagebox
saying it's msiexec.exe's fault.
Even the new version crashes with an identical error: i'll attach its log under
this one.

[Bug 16498] hex workshop crashes in msi

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

--- Comment #4 from Luca Bennati   2009-08-12 10:25:20 ---
Created an attachment (id=23023)
 --> (http://bugs.winehq.org/attachment.cgi?id=23023)
log of the setup crash in HW 6.01

[Bug 16498] hex workshop crashes in msi

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

Louis Lenders  changed:

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

--- Comment #5 from Louis Lenders   2009-11-18 09:49:30 ---
This turns out to be a duplicate of bug 17665 (or actually the other way
around, but the latter bug has a fixing patch now). The patch fixes this bug
too

*** This bug has been marked as a duplicate of bug 17665 ***

[Bug 16498] hex workshop crashes in msi

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

Louis Lenders  changed:

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

--- Comment #6 from Louis Lenders   2009-11-18 09:49:51 ---
closing dup