In current code, PSID is processed as string and the length is 0x20.
Current code only reserved 0x20 length buffer for it, no extra buffer
for the '\0'. When driver call UnicodeStrToAsciiStrS to convert PSID,
it search the '\0' for the end. So extra dirty data saved in PSID
info which caused PSID revert action failed. This patch reserved
extra 1 byte data for the '\0'.
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Eric Dong <eric.dong@intel.com>
Cc: Star Zeng <star.zeng@intel.com>
Reviewed-by: Star Zeng <star.zeng@intel.com>
OPAL_DISK *OpalDisk;\r
TCG_RESULT Ret;\r
OPAL_SESSION Session;\r
+ UINT8 TmpBuf[PSID_CHARACTER_STRING_END_LENGTH];\r
\r
Ret = TcgResultFailure;\r
\r
OpalHiiGetBrowserData();\r
\r
- UnicodeStrToAsciiStrS (gHiiConfiguration.Psid, (CHAR8*)Psid.Psid, PSID_CHARACTER_LENGTH);\r
+ ZeroMem (TmpBuf, sizeof (TmpBuf));\r
+ UnicodeStrToAsciiStrS (gHiiConfiguration.Psid, (CHAR8*)TmpBuf, PSID_CHARACTER_STRING_END_LENGTH);\r
+ CopyMem (Psid.Psid, TmpBuf, PSID_CHARACTER_LENGTH);\r
\r
OpalDisk = HiiGetOpalDiskCB (gHiiConfiguration.SelectedDiskIndex);\r
if (OpalDisk != NULL) {\r
\r
// PSID Length\r
#define PSID_CHARACTER_LENGTH 0x20\r
+#define PSID_CHARACTER_STRING_END_LENGTH 0x21\r
\r
// ID's for various forms that will be used by HII\r
#define FORMID_VALUE_MAIN_MENU 0x01\r
UINT8 KeepUserData;\r
UINT16 AvailableFields;\r
UINT16 Password[MAX_PASSWORD_CHARACTER_LENGTH];\r
- UINT16 Psid[PSID_CHARACTER_LENGTH];\r
+ UINT16 Psid[PSID_CHARACTER_STRING_END_LENGTH];\r
UINT8 EnableBlockSid;\r
} OPAL_HII_CONFIGURATION;\r
#pragma pack()\r