diff options
author | Dimitrije Pavlov <dimitrije.pavlov@arm.com> | 2022-08-16 15:28:35 -0500 |
---|---|---|
committer | mergify[bot] <37929162+mergify[bot]@users.noreply.github.com> | 2022-08-16 20:52:19 +0000 |
commit | 30d62f5e3170c01f3331677894f80415eec92f23 (patch) | |
tree | 5d91013e7d937a660cbb7c44190f74333eff0658 /OvmfPkg/PlatformDxe | |
parent | 2812668bfc121ee792cf3302195176ef4a2ad0bc (diff) | |
download | edk2-30d62f5e3170c01f3331677894f80415eec92f23.zip edk2-30d62f5e3170c01f3331677894f80415eec92f23.tar.gz edk2-30d62f5e3170c01f3331677894f80415eec92f23.tar.bz2 |
OvmfPkg/PlatformDxe: Check ExtractConfig and RouteConfig arguments
The current implementation does not check if Progress or Results
pointers in ExtractConfig are NULL, or if Progress pointer in
RouteConfig is NULL. This causes the SCT test suite to crash.
Add a check to return EFI_INVALID_PARAMETER if any of these pointers
are NULL.
Signed-off-by: Dimitrije Pavlov <Dimitrije.Pavlov@arm.com>
Reviewed-by: Sunny Wang <sunny.wang@arm.com>
Diffstat (limited to 'OvmfPkg/PlatformDxe')
-rw-r--r-- | OvmfPkg/PlatformDxe/Platform.c | 8 |
1 files changed, 8 insertions, 0 deletions
diff --git a/OvmfPkg/PlatformDxe/Platform.c b/OvmfPkg/PlatformDxe/Platform.c index 4bf2271..a6d459f 100644 --- a/OvmfPkg/PlatformDxe/Platform.c +++ b/OvmfPkg/PlatformDxe/Platform.c @@ -232,6 +232,10 @@ ExtractConfig ( DEBUG ((DEBUG_VERBOSE, "%a: Request=\"%s\"\n", __FUNCTION__, Request));
+ if ((Progress == NULL) || (Results == NULL)) {
+ return EFI_INVALID_PARAMETER;
+ }
+
Status = PlatformConfigToFormState (&MainFormState);
if (EFI_ERROR (Status)) {
*Progress = Request;
@@ -340,6 +344,10 @@ RouteConfig ( Configuration
));
+ if (Progress == NULL) {
+ return EFI_INVALID_PARAMETER;
+ }
+
//
// the "read" step in RMW
//
|