fix(minidump_parser): improve subprocess handling

Author: tysmith

src/ffpuppet/minidump_parser.py

@@ -5,6 +5,7 @@
 
 from __future__ import annotations
 
+import sys
 from json import JSONDecodeError, load
 from logging import DEBUG, INFO, basicConfig, getLogger
 from pathlib import Path
@@ -13,6 +14,12 @@
 from tempfile import TemporaryFile, mkdtemp
 from typing import IO, Any
 
+if sys.platform == "win32":
+    # pylint: disable=ungrouped-imports
+    from subprocess import CREATE_NEW_PROCESS_GROUP
+else:
+    CREATE_NEW_PROCESS_GROUP = 0
+
 EXTRA_FIELDS = (
     # android only
     "CrashType",
@@ -233,7 +240,15 @@ def create_log(self, src: Path, filename: str, timeout: int = 300) -> Path:
         ):
             LOG.debug("running '%s'", " ".join(cmd))
             try:
-                run(cmd, check=True, stderr=err_fp, stdout=out_fp, timeout=timeout)
+                run(
+                    cmd,
+                    check=True,
+                    creationflags=CREATE_NEW_PROCESS_GROUP,
+                    start_new_session=sys.platform != "win32",
+                    stderr=err_fp,
+                    stdout=out_fp,
+                    timeout=timeout,
+                )
                 out_fp.seek(0)
                 # load json, format data and write log
                 with dst.open("wb") as log_fp: