Changeset 1714 for trunk/eraser6

Timestamp:

1/28/2010 7:50:15 AM (3 years ago)

Author:

lowjoel

Message:

Fixed race condition potentially created by initialising the remote executor server thread immediately upon construction since Run is not yet called.

File:

• trunk/eraser6/Eraser.Manager/RemoteExecutor.cs (modified) (2 diffs)

trunk/eraser6/Eraser.Manager/RemoteExecutor.cs

@@ -100 +100 @@
             thread = new Thread(Main);
             serverLock = new Semaphore(maxServerInstances, maxServerInstances);
-
-            thread.Start();
-            Thread.Sleep(0);
         }
 
@@ -125 +122 @@
 
             base.Dispose(disposing);
+        }
+
+        public override void Run()
+        {
+            thread.Start();
+            Thread.Sleep(0);
+            base.Run();
         }