OvmfPkg/VirtioFsDxe: Check GetDriverName arguments

The current implementation does not check if Language or DriverName
are 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>

diff --git a/OvmfPkg/VirtioFsDxe/DriverBinding.c b/OvmfPkg/VirtioFsDxe/DriverBinding.c
index 86eb9cf0ba51b72c72838fc7400a3d9cbb337720..69d2bb77708954ef7e10f930af2d1215c267e4ef 100644 (file)
--- a/OvmfPkg/VirtioFsDxe/DriverBinding.c
+++ b/OvmfPkg/VirtioFsDxe/DriverBinding.c
@@ -218,6 +218,10 @@ VirtioFsGetDriverName (
   OUT CHAR16                        **DriverName\r
   )\r
 {\r
+  if ((Language == NULL) || (DriverName == NULL)) {\r
+    return EFI_INVALID_PARAMETER;\r
+  }\r
+\r
   if (AsciiStrCmp (Language, "en") != 0) {\r
     return EFI_UNSUPPORTED;\r
   }\r