reject negative autoboot timeout fix

Signed-off-by: Filip Lewiński <[email protected]>
Dasharo · Oct 26, 2023 · ccfd3da · ccfd3da
1 parent 43e6175
commit ccfd3da
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 3 deletions.
diff --git a/MdeModulePkg/Library/CustomizedDisplayLib/CustomizedDisplayLib.uni b/MdeModulePkg/Library/CustomizedDisplayLib/CustomizedDisplayLib.uni
@@ -43,8 +43,8 @@
                                        #language fr-FR  "<Spacebar>Bascule la Case de pointage"
 #string NV_UPDATE_MESSAGE              #language en-US  "Configuration changed"
                                        #language fr-FR  "Configuration changed"
-#string INPUT_ERROR_MESSAGE            #language en-US  "!!"
-                                       #language fr-FR  "!!"
+#string INPUT_ERROR_MESSAGE            #language en-US  "Invalid input value"
+                                       #language fr-FR  "Invalid input value"
 #string EMPTY_STRING                   #language en-US  ""
                                        #language fr-FR  ""
 #string ARE_YOU_SURE_YES               #language en-US  "Y"

diff --git a/MdeModulePkg/Universal/DisplayEngineDxe/InputHandler.c b/MdeModulePkg/Universal/DisplayEngineDxe/InputHandler.c
@@ -619,7 +619,7 @@ GetNumericInput (
   if ((Question->OpCode->OpCode == EFI_IFR_NUMERIC_OP) && (NumericOp != NULL)) {
     if ((NumericOp->Flags & EFI_IFR_DISPLAY) == EFI_IFR_DISPLAY_UINT_HEX){
       HexInput = TRUE;
-    } else if ((NumericOp->Flags & EFI_IFR_DISPLAY) == 0){
+    } else if ( ((NumericOp->Flags & EFI_IFR_DISPLAY) == 0) || ((NumericOp->Flags & EFI_IFR_DISPLAY) == EFI_IFR_DISPLAY_UINT_DEC) ){
       //
       // Display with EFI_IFR_DISPLAY_INT_DEC type. Support negative number.
       //
@@ -752,6 +752,11 @@ GetNumericInput (
         // In Manual input mode, check whether input the negative flag.
         //
         if (Key.UnicodeChar == '-') {
+          if ((NumericOp->Flags & EFI_IFR_DISPLAY) == EFI_IFR_DISPLAY_UINT_DEC){
+            if (Count == 0) UpdateStatusBar(INPUT_ERROR, TRUE);
+            break;
+          }
+
           if (Negative) {
             break;
           }