Merge pull request #38 from ChainSwordCS/pr/datetime-fix

Fixed an oversight with checking validity of a dirent's datetime in Metadata Analyzer
aerosoul94 · Nov 26, 2024 · ac5c146 · ac5c146
2 parents 99a1937 + 3805d11
commit ac5c146
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/.github/workflows/build.yaml b/.github/workflows/build.yaml
@@ -40,7 +40,7 @@ jobs:
 
     # Upload the release build artifacts
     - name: Upload artifacts
-      uses: actions/upload-artifact@v2
+      uses: actions/upload-artifact@v4
       with:
         name: FATXTools-Release
         path: FATXTools/bin/Release/
diff --git a/FATX/Analyzers/MetadataAnalyzer.cs b/FATX/Analyzers/MetadataAnalyzer.cs
@@ -256,7 +256,7 @@ private bool IsValidDateTime(TimeStamp dateTime)
                 return false;
             }
 
-            if (dateTime.Day > MaxDays[dateTime.Month - 1])
+            if (dateTime.Day > MaxDays[dateTime.Month - 1] || dateTime.Day < 1)
             {
                 return false;
             }