[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/…
==============================================================================
--- 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 */