]> git.proxmox.com Git - mirror_edk2.git/commitdiff
MdeModulePkg/UiApp: Exit function when parameter is unsupported or invalid
authorDandan Bi <dandan.bi@intel.com>
Tue, 17 May 2016 03:25:34 +0000 (11:25 +0800)
committerStar Zeng <star.zeng@intel.com>
Wed, 18 May 2016 13:53:57 +0000 (21:53 +0800)
When the parameter is unsupported or invalid,
should exit the function.

Cc: Qiu Shumin <shumin.qiu@intel.com>
Cc: Eric Dong <eric.dong@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi <dandan.bi@intel.com>
Reviewed-by: Eric Dong <eric.dong@intel.com>
Reviewed-by: Qiu Shumin <shumin.qiu@intel.com>
MdeModulePkg/Application/UiApp/FrontPageCustomizedUiSupport.c

index 80450a5ac47b676fd731c88d33dfb4bd547cce82..b838222106ee956e1bd9a57b4d3a49d8c52950fe 100644 (file)
@@ -195,11 +195,13 @@ UiSupportLibCallbackHandler (
     // Do nothing for other UEFI Action. Only do call back when data is changed.\r
     //\r
     *Status = EFI_UNSUPPORTED;\r
     // Do nothing for other UEFI Action. Only do call back when data is changed.\r
     //\r
     *Status = EFI_UNSUPPORTED;\r
+    return TRUE;\r
   }\r
 \r
   if (Action == EFI_BROWSER_ACTION_CHANGED) {\r
     if ((Value == NULL) || (ActionRequest == NULL)) {\r
       *Status = EFI_INVALID_PARAMETER;\r
   }\r
 \r
   if (Action == EFI_BROWSER_ACTION_CHANGED) {\r
     if ((Value == NULL) || (ActionRequest == NULL)) {\r
       *Status = EFI_INVALID_PARAMETER;\r
+      return TRUE;\r
     }\r
 \r
     *Status = EFI_SUCCESS;\r
     }\r
 \r
     *Status = EFI_SUCCESS;\r