ntdll: send_debug_event: don't send uninitialized bytes (try 3)
dank at kegel.com
Sun Nov 18 11:39:51 CST 2007
Gaah. Typed max() when I mean min(). And then didn't really test.
Fixed the typo, tested, covered self with sackcloth and ashes.
On Nov 16, 2007 4:34 PM, Dan Kegel <dank at kegel.com> wrote:
> This try uses FIELD_OFFSET as suggested by Rob. Also preserves
> the property of the existing code that an insane parameter
> count doesn't cause a buffer overrun. (That might not
> really be needed, I'm just being paranoid. If it really is needed,
> maybe RtlRaiseException should check its parameters
> more strictly.)
> This patch is a slight optimization; it avoids sending
> unused and uninitialized parts of the exception record
> to the server. (This is the other side of
> Both stand alone, they don't need to go together.)
> Found via Valgrind warning:
> Syscall param writev(vector) points to uninitialised byte(s)
> at 0x40007F2: (within /lib/ld-2.5.so)
> by 0x44087F4: wine_server_call (server.c:244)
> by 0x43DFF48: send_debug_event (exception.c:207)
> by 0x43E01FE: raise_exception (exception.c:364)
> by 0x43E05A8: __regs_RtlRaiseException (exception.c:395)
> by 0x4409A3B: raise_segv_exception (signal_i386.c:1030)
> by 0xDEADBABD: ???
> by 0x676CAB2: DOSMEM_InitDosMemory (dosmem.c:172)
> by 0x676CF31: DllMain (dosvm.c:708)
> when installing Word 97 Viewer, but probably comes up
> in many places.
Wine for Windows ISVs: http://kegel.com/wine/isv
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 1064 bytes
Desc: not available
Url : http://www.winehq.org/pipermail/wine-patches/attachments/20071118/83ba8540/attachment.bin
More information about the wine-patches