[User32]
authorJames Tabor <james.tabor@reactos.org>
Mon, 14 Apr 2014 06:05:34 +0000 (06:05 +0000)
committerJames Tabor <james.tabor@reactos.org>
Mon, 14 Apr 2014 06:05:34 +0000 (06:05 +0000)
- Fixed get menu info, check parameters first than the handle. See CORE-8064.

svn path=/trunk/; revision=62747

reactos/win32ss/user/user32/windows/menu.c

index c051f69..d3eeae8 100644 (file)
@@ -4269,15 +4269,15 @@ GetMenuInfo(HMENU hmenu,
   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);
   mi.fMask = lpcmi->fMask;