OvmfPkg/VirtioFsDxe: call IsTimeValid() before EfiTimeToEpoch()

[mirror_edk2.git] / OvmfPkg / VirtioFsDxe / Helpers.c

diff --git a/OvmfPkg/VirtioFsDxe/Helpers.c b/OvmfPkg/VirtioFsDxe/Helpers.c
index 443bbdc616ac039bead49dd1847b5608cff3222e..b81c04e0a4e81df5d72f99d164c8f49d6784e15d 100644 (file)
--- a/OvmfPkg/VirtioFsDxe/Helpers.c
+++ b/OvmfPkg/VirtioFsDxe/Helpers.c
@@ -2244,12 +2244,19 @@ VirtioFsGetFuseSizeUpdate (
                            since the Epoch). Otherwise, Mtime is not written\r
                            to.\r
 \r
-  @retval EFI_SUCCESS        Output parameters have been set successfully.\r
-\r
-  @retval EFI_ACCESS_DENIED  NewInfo requests changing both CreateTime and\r
-                             ModificationTime, but to values that differ from\r
-                             each other. The Virtio Filesystem device does not\r
-                             support this.\r
+  @retval EFI_SUCCESS            Output parameters have been set successfully.\r
+\r
+  @retval EFI_INVALID_PARAMETER  At least one of the CreateTime, LastAccessTime\r
+                                 and ModificationTime fields in NewInfo\r
+                                 represents an actual update relative to the\r
+                                 current state of the file (expressed in Info),\r
+                                 but does not satisfy the UEFI spec\r
+                                 requirements on EFI_TIME.\r
+\r
+  @retval EFI_ACCESS_DENIED      NewInfo requests changing both CreateTime and\r
+                                 ModificationTime, but to values that differ\r
+                                 from each other. The Virtio Filesystem device\r
+                                 does not support this.\r
 **/\r
 EFI_STATUS\r
 VirtioFsGetFuseTimeUpdates (\r
@@ -2285,6 +2292,9 @@ VirtioFsGetFuseTimeUpdates (
         CompareMem (NewTime[Idx], Time[Idx], sizeof (EFI_TIME)) == 0) {\r
       Change[Idx] = FALSE;\r
     } else {\r
+      if (!IsTimeValid (NewTime[Idx])) {\r
+        return EFI_INVALID_PARAMETER;\r
+      }\r
       Change[Idx] = TRUE;\r
       Seconds[Idx] = EfiTimeToEpoch (NewTime[Idx]);\r
     }\r