diff options
| author | Amit Pundir <amit.pundir@linaro.org> | 2022-05-30 20:00:44 +0530 |
|---|---|---|
| committer | Amit Pundir <amit.pundir@linaro.org> | 2022-05-31 19:43:25 +0530 |
| commit | 3d02c502696169104aec0325a7c3dff98f487ab3 (patch) | |
| tree | 01a3d8ac49bf637b926419b6406e7eb6cb18d709 | |
| parent | 212cd030a7c5bc95cecf8d9693790ab97cdd5cb0 (diff) | |
QcomModulePkg: Bootconfig: Use memcmp instead of AsciiStrnCmpv4_header
AsciiStrnCmp() asserts on string length when we try to
compare a non-null terminated string:
DXE_ASSERT!: .../MdePkg/Library/BaseLib/String.c (1237): Length < _gPcd_FixedAtBuild_PcdMaximumAsciiStringLength
So use memcmp() to compare bootconfig Magic instead.
Suggested-by: Bjorn Andersson <bjorn.andersson@linaro.org
Signed-off-by: Amit Pundir <amit.pundir@linaro.org>
| -rw-r--r-- | QcomModulePkg/Library/BootLib/Bootconfig.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/QcomModulePkg/Library/BootLib/Bootconfig.c b/QcomModulePkg/Library/BootLib/Bootconfig.c index 093768b6f1..41e5ebdf6d 100644 --- a/QcomModulePkg/Library/BootLib/Bootconfig.c +++ b/QcomModulePkg/Library/BootLib/Bootconfig.c @@ -57,7 +57,7 @@ CHAR8 Magic[] = "#BOOTCONFIG\n"; */ static BOOLEAN IsTrailerPresent (UINT64 BootconfigEndAddr) { - return !AsciiStrnCmp ((CHAR8*)(BootconfigEndAddr - V4_BOOTCONFIG_MAGIC_SIZE) + return !memcmp ((CHAR8*)(BootconfigEndAddr - V4_BOOTCONFIG_MAGIC_SIZE) , Magic, V4_BOOTCONFIG_MAGIC_SIZE); } /* |