[FRAMEBUF] Correctly check the return value of the EngDeviceIoControl() call and...

diff --git a/win32ss/drivers/displays/framebuf/surface.c b/win32ss/drivers/displays/framebuf/surface.c
index 43b3b89..b0b74da 100644 (file)
--- a/win32ss/drivers/displays/framebuf/surface.c
+++ b/win32ss/drivers/displays/framebuf/surface.c
@@ -174,27 +174,28 @@ DrvAssertMode(
 
    if (bEnable)
    {
-      BOOLEAN Result;
       /*
        * Reinitialize the device to a clean state.
        */
-      Result = EngDeviceIoControl(ppdev->hDriver, IOCTL_VIDEO_SET_CURRENT_MODE,
-                                  &(ppdev->ModeIndex), sizeof(ULONG), NULL, 0,
-                                  &ulTemp);
+      if (EngDeviceIoControl(ppdev->hDriver, IOCTL_VIDEO_SET_CURRENT_MODE,
+                             &(ppdev->ModeIndex), sizeof(ULONG), NULL, 0,
+                             &ulTemp))
+      {
+          /* We failed, bail out */
+          return FALSE;
+      }
       if (ppdev->BitsPerPixel == 8)
       {
             IntSetPalette(dhpdev, ppdev->PaletteEntries, 0, 256);
       }
 
-      return Result;
-
+      return TRUE;
    }
    else
    {
       /*
        * Call the miniport driver to reset the device to a known state.
        */
-
       return !EngDeviceIoControl(ppdev->hDriver, IOCTL_VIDEO_RESET_DEVICE,
                                  NULL, 0, NULL, 0, &ulTemp);
    }