From a0c91cb895c3181506eee31609ac0a98e5d6288a Mon Sep 17 00:00:00 2001
From: Rutherther <fandabohacek@gmail.com>
Date: Wed, 15 Feb 2023 17:17:48 +0100
Subject: [PATCH] fix: handle exceptions from IsProcessNostaleProcess

---
 src/PacketLogger/Models/NostaleProcesses.cs | 41 +++++++++++----------
 1 file changed, 21 insertions(+), 20 deletions(-)

diff --git a/src/PacketLogger/Models/NostaleProcesses.cs b/src/PacketLogger/Models/NostaleProcesses.cs
index 242adf5215d626ba342e2f8a311f9876db04e274..65f3dd9ff2972994d427f1c448c743ade3886218 100644
--- a/src/PacketLogger/Models/NostaleProcesses.cs
+++ b/src/PacketLogger/Models/NostaleProcesses.cs
@@ -95,17 +95,16 @@ public class NostaleProcesses : IDisposable
         try
         {
             process = Process.GetProcessById(processId);
+            if (!NosBrowserManager.IsProcessNostaleProcess(process))
+            {
+                return;
+            }
         }
         catch (Exception)
         {
             return;
         }
 
-        if (!NosBrowserManager.IsProcessNostaleProcess(process))
-        {
-            return;
-        }
-
         NosBrowserManager nosBrowserManager = new NosBrowserManager
         (
             process,
@@ -126,13 +125,14 @@ public class NostaleProcesses : IDisposable
         if (nosBrowserManager.IsModuleLoaded<PlayerManager>())
         {
             RxApp.MainThreadScheduler.Schedule
-                (() =>
-                    {
-                        _semaphore.Wait();
-                        Processes.Add(new NostaleProcess(process, nosBrowserManager));
-                        _semaphore.Release();
-                    }
-                );
+            (
+                () =>
+                {
+                    _semaphore.Wait();
+                    Processes.Add(new NostaleProcess(process, nosBrowserManager));
+                    _semaphore.Release();
+                }
+            );
         }
         else
         {
@@ -152,14 +152,15 @@ public class NostaleProcesses : IDisposable
             if (process is not null)
             {
                 RxApp.MainThreadScheduler.Schedule
-                    (() =>
-                        {
-                            process.ObserveChanges();
-                            _semaphore.Wait();
-                            Processes.Remove(process);
-                            _semaphore.Release();
-                        }
-                    );
+                (
+                    () =>
+                    {
+                        process.ObserveChanges();
+                        _semaphore.Wait();
+                        Processes.Remove(process);
+                        _semaphore.Release();
+                    }
+                );
             }
         }
     }