X-Git-Url: https://code.delx.au/refind/blobdiff_plain/2cc79ea9b33424d5fc32f15683adf7c956656b9b..add96ff8057d0c03026395992d9ecde081c8dca7:/refind/menu.c

diff --git a/refind/menu.c b/refind/menu.c
index e55bcf4..63bf9fc 100644
--- a/refind/menu.c
+++ b/refind/menu.c
@@ -445,38 +445,38 @@ static UINTN RunGenericMenu(IN REFIT_MENU_SCREEN *Screen, IN MENU_STYLE_FUNC Sty
                 MenuExit = MENU_EXIT_TIMEOUT;
                 break;
             } else if (HaveTimeout || GlobalConfig.ScreensaverTime > 0) {
-		EFI_EVENT           TimerEvent;
-		UINTN		    ElapsCount = 1;
-
-		Status = refit_call5_wrapper(BS->CreateEvent, EVT_TIMER, 0, NULL, NULL, &TimerEvent);
-		if (EFI_ERROR(Status)) {
-		    refit_call1_wrapper(BS->Stall, 100000); // Pause for 100 ms
-		} else {
-		    EFI_EVENT           WaitList[2];
-		    UINTN               Index;
-
-		    refit_call3_wrapper(BS->SetTimer, TimerEvent, TimerRelative, 10000000); // 1s Timeout
-		    WaitList[0] = ST->ConIn->WaitForKey;
-		    WaitList[1] = TimerEvent;
-		    Status = refit_call3_wrapper(BS->WaitForEvent, 2, WaitList, &Index);
-		    refit_call1_wrapper(BS->CloseEvent, TimerEvent);
-		    if (EFI_ERROR(Status))
-			refit_call1_wrapper(BS->Stall, 100000); // Pause for 100 ms
-		    else if(Index == 0)
-			continue;
-		    else
-			ElapsCount = 10; // always counted as 1s to end of the timeout
-		}
-		TimeSinceKeystroke += ElapsCount;
-		if(HaveTimeout) {
-		    TimeoutCountdown = TimeoutCountdown <= ElapsCount ? 0 : TimeoutCountdown - ElapsCount;
-		} else if (GlobalConfig.ScreensaverTime > 0 &&
-			TimeSinceKeystroke > (GlobalConfig.ScreensaverTime * 10))
-		{
-			SaveScreen();
-			State.PaintAll = TRUE;
-			TimeSinceKeystroke = 0;
-		} // if
+                EFI_EVENT           TimerEvent;
+                UINTN               ElapsCount = 1;
+
+                Status = refit_call5_wrapper(BS->CreateEvent, EVT_TIMER, 0, NULL, NULL, &TimerEvent);
+                if (EFI_ERROR(Status)) {
+                    refit_call1_wrapper(BS->Stall, 100000); // Pause for 100 ms
+                } else {
+                    EFI_EVENT           WaitList[2];
+                    UINTN               Index;
+
+                    refit_call3_wrapper(BS->SetTimer, TimerEvent, TimerRelative, 10000000); // 1s Timeout
+                    WaitList[0] = ST->ConIn->WaitForKey;
+                    WaitList[1] = TimerEvent;
+                    Status = refit_call3_wrapper(BS->WaitForEvent, 2, WaitList, &Index);
+                    refit_call1_wrapper(BS->CloseEvent, TimerEvent);
+                    if (EFI_ERROR(Status))
+                        refit_call1_wrapper(BS->Stall, 100000); // Pause for 100 ms
+                    else if(Index == 0)
+                        continue;
+                    else
+                        ElapsCount = 10; // always counted as 1s to end of the timeout
+                }
+                TimeSinceKeystroke += ElapsCount;
+                if(HaveTimeout) {
+                    TimeoutCountdown = TimeoutCountdown <= ElapsCount ? 0 : TimeoutCountdown - ElapsCount;
+                } else if (GlobalConfig.ScreensaverTime > 0 &&
+                        TimeSinceKeystroke > (GlobalConfig.ScreensaverTime * 10))
+                {
+                        SaveScreen();
+                        State.PaintAll = TRUE;
+                        TimeSinceKeystroke = 0;
+                } // if
             } else {
                 refit_call3_wrapper(BS->WaitForEvent, 1, &ST->ConIn->WaitForKey, &index);
             }
@@ -761,12 +761,13 @@ static UINT8 AverageBrightness(EG_IMAGE *Image) {
    UINTN i;
    UINTN Sum = 0;
 
-   if (Image != NULL) {
+   if ((Image != NULL) && ((Image->Width * Image->Height) != 0)) {
       for (i = 0; i < (Image->Width * Image->Height); i++) {
          Sum += (Image->PixelData[i].r + Image->PixelData[i].g + Image->PixelData[i].b);
       }
+      Sum /= (Image->Width * Image->Height * 3);
    } // if
-   return (UINT8) (Sum / (Image->Width * Image->Height * 3));
+   return (UINT8) Sum;
 } // UINT8 AverageBrightness()
 
 // Display text against the screen's background image. Special case: If Text is NULL
@@ -1051,7 +1052,7 @@ static VOID PaintIcon(IN EG_EMBEDDED_IMAGE *BuiltInIcon, IN CHAR16 *ExternalFile
    }
 } // static VOID ()
 
-inline UINTN ComputeRow0PosY(VOID) {
+UINTN ComputeRow0PosY(VOID) {
    return ((UGAHeight / 2) - TileSizes[0] / 2);
 } // UINTN ComputeRow0PosY()