Refine code to avoid potential access violation.
Signed-off-by:ydong10 Reviewed-by:lgao4 git-svn-id: https://edk2.svn.sourceforge.net/svnroot/edk2/trunk/edk2@11976 6f19259b-4bc3-4df7-8a09-765794883524
This commit is contained in:
@@ -1168,7 +1168,7 @@ ProcessCallBackFunction (
|
|||||||
// "retrieve" should update to the question's temp buffer.
|
// "retrieve" should update to the question's temp buffer.
|
||||||
//
|
//
|
||||||
if (Action == EFI_BROWSER_ACTION_CHANGING || Action == EFI_BROWSER_ACTION_RETRIEVE) {
|
if (Action == EFI_BROWSER_ACTION_CHANGING || Action == EFI_BROWSER_ACTION_RETRIEVE) {
|
||||||
SetQuestionValue(Selection->FormSet,Selection->Form, Question,TRUE);
|
SetQuestionValue(Selection->FormSet, Selection->Form, Statement, TRUE);
|
||||||
}
|
}
|
||||||
} else if (Status == EFI_UNSUPPORTED) {
|
} else if (Status == EFI_UNSUPPORTED) {
|
||||||
//
|
//
|
||||||
|
@@ -2877,6 +2877,7 @@ UiDisplayMenu (
|
|||||||
AdjustDateAndTimePosition (TRUE, &NewPos);
|
AdjustDateAndTimePosition (TRUE, &NewPos);
|
||||||
if (NewPos->BackLink != &gMenuOption) {
|
if (NewPos->BackLink != &gMenuOption) {
|
||||||
MenuOption = MENU_OPTION_FROM_LINK (NewPos);
|
MenuOption = MENU_OPTION_FROM_LINK (NewPos);
|
||||||
|
ASSERT (MenuOption != NULL);
|
||||||
NewLine = TRUE;
|
NewLine = TRUE;
|
||||||
NewPos = NewPos->BackLink;
|
NewPos = NewPos->BackLink;
|
||||||
|
|
||||||
@@ -2888,7 +2889,6 @@ UiDisplayMenu (
|
|||||||
}
|
}
|
||||||
NextMenuOption = MENU_OPTION_FROM_LINK (NewPos);
|
NextMenuOption = MENU_OPTION_FROM_LINK (NewPos);
|
||||||
|
|
||||||
ASSERT (MenuOption != NULL);
|
|
||||||
if (Difference < 0) {
|
if (Difference < 0) {
|
||||||
//
|
//
|
||||||
// We hit the begining MenuOption that can be focused
|
// We hit the begining MenuOption that can be focused
|
||||||
|
Reference in New Issue
Block a user