fix race conditions

Author: elcritch

atlas.nimble

@@ -1,5 +1,5 @@
 # Package
-version = "0.14.4"
+version = "0.14.5"
 author = "Araq"
 description = "Atlas is a simple package cloner tool. It manages an isolated project."
 license = "MIT"

config.nims

@@ -15,6 +15,7 @@ else:
 --path:"$nim"
 --nimcache:".nimcache"
 --d:ssl
+--d:useMalloc
 --gc:atomicArc
 --threads:on
 

src/basic/gitprogresspool.nim

@@ -53,6 +53,7 @@ type
   GitProgressWorkerArgs = object
     jobs: seq[IndexedGitProgressJob]
     events: ptr GitProgressEventQueue
+    startLock: ptr Lock
 
   GitProgressJobState = object
     label: string
@@ -380,13 +381,19 @@ proc gitProgressWorker(args: GitProgressWorkerArgs) {.thread.} =
       label: job.label
     )
 
-    var process = startProcess(
-      job.command,
-      workingDir = job.workingDir,
-      args = job.args,
-      options = {poUsePath, poStdErrToStdOut}
-    )
+    var process: Process
     try:
+      acquire(args.startLock[])
+      try:
+        process = startProcess(
+          job.command,
+          workingDir = job.workingDir,
+          args = job.args,
+          options = {poUsePath, poStdErrToStdOut}
+        )
+      finally:
+        release(args.startLock[])
+
       let readResult = readGitProgressOutput(process, indexedJob.index, job.label, args.events)
       let exitCode = waitForExit(process)
       args.events.pushEvent GitProgressEvent(
@@ -397,8 +404,22 @@ proc gitProgressWorker(args: GitProgressWorkerArgs) {.thread.} =
         exitCode: exitCode,
         progress: readResult.progress
       )
+    except CatchableError as exc:
+      args.events.pushEvent GitProgressEvent(
+        kind: gpFinished,
+        jobIndex: indexedJob.index,
+        label: job.label,
+        output: $exc.name & ": " & exc.msg,
+        exitCode: 1,
+        progress: GitProgressSnapshot(phase: "failed", percent: -1)
+      )
     finally:
-      close(process)
+      if process != nil:
+        acquire(args.startLock[])
+        try:
+          close(process)
+        finally:
+          release(args.startLock[])
 
 proc runGitProgressJobs*(
     jobs: openArray[GitProgressJob];
@@ -423,14 +444,20 @@ proc runGitProgressJobs*(
   let renderInteractive = interactiveProgressEnabled(showProgress)
   var eventQueue: GitProgressEventQueue
   initLock(eventQueue.lock)
+  var startLock: Lock
+  initLock(startLock)
 
   var workerSlices = splitJobs(jobsSeq, effectiveWorkers)
   var threads = newSeq[Thread[GitProgressWorkerArgs]](effectiveWorkers)
   for idx in 0..<effectiveWorkers:
     createThread(
       threads[idx],
       gitProgressWorker,
-      GitProgressWorkerArgs(jobs: workerSlices[idx], events: addr eventQueue)
+      GitProgressWorkerArgs(
+        jobs: workerSlices[idx],
+        events: addr eventQueue,
+        startLock: addr startLock
+      )
     )
 
   var finished = 0
@@ -501,3 +528,4 @@ proc runGitProgressJobs*(
     stdout.flushFile()
 
   deinitLock(eventQueue.lock)
+  deinitLock(startLock)