From 163dc5e0b285c9d632ad75b5b447118288c8a151 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Daniel=20K=C3=B6hntopp?=
 <33926917+Archiatrus@users.noreply.github.com>
Date: Wed, 28 Nov 2018 20:40:40 +0100
Subject: [PATCH] Prevent TerminateProcess(0)

In a proxy setting the process_id usually referring to the game instances is default initialized to 0. On Linux based systems `TerminateProcess(0)` will kill all processes within the bot's processing group, including the bot itself.
---
 src/sc2api/sc2_coordinator.cc | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/src/sc2api/sc2_coordinator.cc b/src/sc2api/sc2_coordinator.cc
index cbe970e4..3ce44635 100644
--- a/src/sc2api/sc2_coordinator.cc
+++ b/src/sc2api/sc2_coordinator.cc
@@ -197,7 +197,9 @@ CoordinatorImp::CoordinatorImp() :
 
 CoordinatorImp::~CoordinatorImp() {
     for (auto& p : process_settings_.process_info) {
-        TerminateProcess(p.process_id);
+        if (p.process_id > 0) {
+            TerminateProcess(p.process_id);
+        }
     }
 }