[Bug 14499] New: ImageDirectoryEntryToDataEx: section header param [out, optional], needs to be zeroed before RtlImageRvaToVa

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

Summary: ImageDirectoryEntryToDataEx: section header param [out,
                    optional], needs to be zeroed before RtlImageRvaToVa
           Product: Wine
           Version: CVS/GIT
          Platform: PC
        OS/Version: Linux
            Status: UNCONFIRMED
          Severity: normal
          Priority: P2
         Component: dbghelp
        AssignedTo: [e-mail]
        ReportedBy: [e-mail]

Hello,

a quickie, as title says.
Section header param passed to ImageDirectoryEntryToDataEx is [out, optional].
Crashes RtlImageRvaToVa if caller doesn't initialize the out param to zero
(stack var).

--- snip ---
..
0020:Call
dbghelp.ImageDirectoryEntryToDataEx(00340000,00000000,00000006,0032cc34,0032cc38)
ret=004048ec
0020:Call ntdll.RtlImageNtHeader(00340000) ret=6076ce7b
0020:Ret  ntdll.RtlImageNtHeader() retval=003400e8 ret=6076ce7b
0020:Call ntdll.RtlImageRvaToVa(003400e8,00340000,00001270,0032cc38)
ret=6076ceeb
0020:trace:seh:raise_exception code=c0000005 flags=0 addr=0x7bc4337a
--- snip ---

Fix: unconditionally "if (section) *section = NULL" on
ImageDirectoryEntryToDataEx entry (like size).

Regards

[Bug 14499] ImageDirectoryEntryToDataEx: section header param [out, optional], needs to be zeroed before RtlImageRvaToVa

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

Anastasius Focht  changed:

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

--- Comment #1 from Anastasius Focht   2008-10-27 05:40:41 ---
Hello,

bug was fixed by commit 97429b32f80773f0b77d4a0d273ba520defa3f2a

Regards

[Bug 14499] ImageDirectoryEntryToDataEx: section header param [out, optional], needs to be zeroed before RtlImageRvaToVa

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

Alexandre Julliard  changed:

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

--- Comment #2 from Alexandre Julliard   2008-11-07 10:46:55 ---
Closing bugs fixed in 1.1.8.

[Bug 14499] ImageDirectoryEntryToDataEx: section header param [out, optional], needs to be zeroed before RtlImageRvaToVa

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

Austin English  changed:

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

[Bug 14499] ImageDirectoryEntryToDataEx: section header param [out, optional], needs to be zeroed before RtlImageRvaToVa

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

Anastasius Focht  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
      Fixed by SHA1|                            |97429b32f80773f0b77d4a0d273
                   |                            |ba520defa3f2a
            Version|unspecified                 |1.1.1

--- Comment #3 from Anastasius Focht  2011-10-12 03:36:12 CDT ---
Hello,

filling/correcting fields ...

Regards