[ros-diffs] [pschweitzer] 66354: [KERNEL32] Don't pretend that ExceptionRecord.ExceptionInformation[2] is a pointer to a string in case of Delphi exception. This can lead to invalid pointer dereference in DbgP...

Author: pschweitzer
Date: Thu Feb 19 18:36:32 2015
New Revision: 66354

URL: http://svn.reactos.org/svn/reactos?rev=66354&view=rev
Log:
[KERNEL32]
Don't pretend that ExceptionRecord.ExceptionInformation[2] is a pointer to a string in case of Delphi exception. This can lead to invalid pointer dereference in DbgPrint then.

For instance:
(../../dll/win32/kernel32/client/except.c:371) Delphi Exception at address: 50075885
(../../dll/win32/kernel32/client/except.c:372) Exception-Object: 00E885A8
(../../dll/win32/kernel32/client/except.c:373) Exception text: 2

Modified:
    trunk/reactos/dll/win32/kernel32/client/except.c

Modified: trunk/reactos/dll/win32/kernel32/client/except.c
URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/kernel32/client/e...
==============================================================================
--- trunk/reactos/dll/win32/kernel32/client/except.c	[iso-8859-1] (original)
+++ trunk/reactos/dll/win32/kernel32/client/except.c	[iso-8859-1] Thu Feb 19 18:36:32 2015
@@ -370,7 +370,7 @@
     {
         DPRINT1("Delphi Exception at address: %p\n", ExceptionRecord.ExceptionInformation[0]);
         DPRINT1("Exception-Object: %p\n", ExceptionRecord.ExceptionInformation[1]);
-        DPRINT1("Exception text: %s\n", ExceptionRecord.ExceptionInformation[2]);
+        DPRINT1("Exception text: %x\n", ExceptionRecord.ExceptionInformation[2]);
     }

/* Trace the wine special error and show the modulename and functionname */