mirror of https://github.com/acidanthera/audk.git
MdeModulePkg/SetupBrowser:Don't support password without interactive flag
In current SetupBrowser, the logic related to non-interative password is not correct. How to support it correctly or whether support it is still under investigation. First step remove the incorrect logic. Cc: Liming Gao <liming.gao@intel.com> Cc: Eric Dong <eric.dong@intel.com> Cc: Jiewen Yao <jiewen.yao@intel.com> Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi <dandan.bi@intel.com> Reviewed-by: Liming Gao <liming.gao@intel.com>
This commit is contained in:
parent
88f0c4e29c
commit
9e2462b8a2
|
@ -6108,29 +6108,10 @@ PasswordCheck (
|
|||
return EFI_UNSUPPORTED;
|
||||
}
|
||||
} else {
|
||||
if (PasswordString == NULL) {
|
||||
return EFI_SUCCESS;
|
||||
}
|
||||
|
||||
//
|
||||
// Check whether has preexisted password.
|
||||
// If a password doesn't have the CALLBACK flag, browser will not handle it.
|
||||
//
|
||||
if (PasswordString[0] == 0) {
|
||||
if (*((CHAR16 *) Question->BufferValue) == 0) {
|
||||
return EFI_SUCCESS;
|
||||
} else {
|
||||
return EFI_NOT_READY;
|
||||
}
|
||||
}
|
||||
|
||||
//
|
||||
// Check whether the input password is same as preexisted password.
|
||||
//
|
||||
if (StrnCmp (PasswordString, (CHAR16 *) Question->BufferValue, Question->StorageWidth/sizeof (CHAR16)) == 0) {
|
||||
return EFI_SUCCESS;
|
||||
} else {
|
||||
return EFI_NOT_READY;
|
||||
}
|
||||
return EFI_UNSUPPORTED;
|
||||
}
|
||||
|
||||
//
|
||||
|
|
Loading…
Reference in New Issue