To fix,
authorxgu3 <xgu3@de2fecce-e211-0410-80a6-f3fac2684e05>
Fri, 12 Jan 2007 10:27:54 +0000 (10:27 +0000)
committerxgu3 <xgu3@de2fecce-e211-0410-80a6-f3fac2684e05>
Fri, 12 Jan 2007 10:27:54 +0000 (10:27 +0000)
1662 Graphics Console Driver: No checking for invalid parameter in SetMode()
8588 Graphic Console should not chech PixelFormat for 800*600 resolution

git-svn-id: https://edk2.tianocore.org/svn/edk2/trunk@2242 de2fecce-e211-0410-80a6-f3fac2684e05

edk2/EdkModulePkg/Universal/Console/GraphicsConsole/Dxe/GraphicsConsole.c

index 40e89a9..05edbd0 100644 (file)
@@ -378,9 +378,7 @@ GraphicsConsoleControllerDriverStart (
                          );\r
       if (!EFI_ERROR (Status)) {\r
         if ((Info->HorizontalResolution == 800) &&\r
-            (Info->VerticalResolution == 600) &&\r
-            ((Info->PixelFormat == PixelRedGreenBlueReserved8BitPerColor) ||\r
-             (Info->PixelFormat == PixelBlueGreenRedReserved8BitPerColor))) {\r
+            (Info->VerticalResolution == 600)) {\r
           Status = Private->GraphicsOutput->SetMode (Private->GraphicsOutput, ModeNumber);\r
           if (!EFI_ERROR (Status)) {\r
             gBS->FreePool (Info);\r
@@ -1164,6 +1162,10 @@ GraphicsConsoleConOutSetMode (
   UgaDraw   = Private->UgaDraw;\r
   ModeData  = &(Private->ModeData[ModeNumber]);\r
 \r
+  if (ModeData->Columns <= 0 && ModeData->Rows <= 0) {\r
+    return EFI_UNSUPPORTED;\r
+  }\r
+\r
   //\r
   // Make sure the requested mode number is supported\r
   //\r