[fireball] 59554: [RTL] - Victor Martinez Calvo: Add a missing status check in RtlpQueryRemoteProcessModules. - Aleksey Bragin: The code is quite ugly, adding this check makes it just a little bit ...

Author: fireball
Date: Mon Jul 22 13:45:56 2013
New Revision: 59554

URL: http://svn.reactos.org/svn/reactos?rev=59554&view=rev
Log:
[RTL]
- Victor Martinez Calvo: Add a missing status check in RtlpQueryRemoteProcessModules.
- Aleksey Bragin: The code is quite ugly, adding this check makes it just a little bit less uglier :-)

CORE-7059 #resolve #comment Committed in 59554, thanks.

Modified:
    trunk/reactos/lib/rtl/dbgbuffer.c

Modified: trunk/reactos/lib/rtl/dbgbuffer.c
URL: http://svn.reactos.org/svn/reactos/trunk/reactos/lib/rtl/dbgbuffer.c?rev=595...
==============================================================================
--- trunk/reactos/lib/rtl/dbgbuffer.c	[iso-8859-1] (original)
+++ trunk/reactos/lib/rtl/dbgbuffer.c	[iso-8859-1] Mon Jul 22 13:45:56 2013
@@ -159,6 +159,13 @@
                                      sizeof(LDR_DATA_TABLE_ENTRY),
                                      NULL);

+        if (!NT_SUCCESS(Status))
+        {
+            /* failure */
+            DPRINT( "NtReadVirtualMemory 3 0x%lx \n", Status);
+            return Status;
+        }
+
         /* Import module name from remote Process user space. */
         Unicode.Length = lmModule.FullDllName.Length;
         Unicode.MaximumLength = lmModule.FullDllName.MaximumLength;