Fix service crash if EC date parsing fails

This is a workaround for issue #61, where certain EC firmware dates appear to be causing FormatExceptions which crashes the YAMDCC service.
Sparronator9999 · Feb 15, 2025 · 619b9e8 · 619b9e8
1 parent 6082b92
commit 619b9e8
Showing 1 changed file with 11 additions and 3 deletions.
diff --git a/YAMDCC.Service/FanControlService.cs b/YAMDCC.Service/FanControlService.cs
@@ -120,10 +120,18 @@ protected override void OnStart(string[] args)
             }
             if (_EC.ReadString(0xAC, 0x10, out string ecDate) && ecDate.Length == 0x10)
             {
-                string temp = $"{ecDate.Substring(4, 4)}-{ecDate.Substring(0, 2)}-{ecDate.Substring(2, 2)}" +
+                try
+                {
+                    string temp = $"{ecDate.Substring(4, 4)}-{ecDate.Substring(0, 2)}-{ecDate.Substring(2, 2)}" +
                     $"T{ecDate.Substring(8, 2)}:{ecDate.Substring(11, 2)}:{ecDate.Substring(14, 2)}";
-                EcInfo.Date = DateTime.ParseExact(temp, "s", CultureInfo.InvariantCulture);
-                Log.Debug($"EC firmware date: {EcInfo.Date:G}");
+                    EcInfo.Date = DateTime.ParseExact(temp, "s", CultureInfo.InvariantCulture);
+                    Log.Debug($"EC firmware date: {EcInfo.Date:G}");
+                }
+                catch (FormatException ex)
+                {
+                    Log.Error($"Failed to parse EC firmware date: {ex.Message}");
+                    Log.Debug($"EC firmware date (raw): {ecDate}");
+                }
             }
         }