[ros-diffs] [jimtabor] 62747: [User32] - Fixed get menu info, check parameters first than the handle. See CORE-8064.

[jimtabor]

Author: jimtabor
Date: Mon Apr 14 06:05:34 2014
New Revision: 62747

URL: http://svn.reactos.org/svn/reactos?rev=62747&view=rev
Log:
[User32]
- Fixed get menu info, check parameters first than the handle. See CORE-8064.

Modified:
    trunk/reactos/win32ss/user/user32/windows/menu.c

Modified: trunk/reactos/win32ss/user/user32/windows/menu.c
URL: 
http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/user/user32/windows/menu.c?rev=62747&r1=62746&r2=62747&view=diff
==============================================================================
--- trunk/reactos/win32ss/user/user32/windows/menu.c    [iso-8859-1] (original)
+++ trunk/reactos/win32ss/user/user32/windows/menu.c    [iso-8859-1] Mon Apr 14 
06:05:34 2014
@@ -4269,14 +4269,14 @@
   BOOL res = FALSE;
   PVOID pMenu;
 
-  if (!(pMenu = ValidateHandle(hmenu, TYPE_MENU)))
-     return FALSE;
-
   if (!lpcmi || (lpcmi->cbSize != sizeof(MENUINFO)))
   {
      SetLastError(ERROR_INVALID_PARAMETER);
      return FALSE;
   }
+
+  if (!(pMenu = ValidateHandle(hmenu, TYPE_MENU)))
+     return FALSE;
 
   RtlZeroMemory(&mi, sizeof(MENUINFO));
   mi.cbSize = sizeof(MENUINFO);