Update subprocess calls from v1.8.8

Noriben.py

@@ -14,6 +14,9 @@
 #           Remove unneeded or obtuse options
 #       Move everything to all major settings to a configuration file
 #
+# Version 1.8.8 - 20 Jan 23
+#       Replaced subprocess .wait() to .communicate() due to known process exec issue
+#       which causes deadlocks in a small amount of cases
 # Version 1.8.7 - 30 Aug 22
 #       Replaced csv.reader with csv.DictReader to have better forward and backward
 #       compatibility. Small changes in style, PEP8
@@ -698,8 +701,9 @@ def process_pml_to_csv(procmonexe, pml_file, pmc_file, csv_file):
     if use_pmc and file_exists(pmc_file):
         cmdline += ' /LoadConfig "{}"'.format(pmc_file)
     log_debug('[*] Running cmdline: {}'.format(cmdline))
-    stdnull = subprocess.Popen(cmdline)
-    stdnull.wait()
+    process = subprocess.Popen(cmdline, stdout=subprocess.PIPE,
+                               stderr=subprocess.PIPE)
+    stdout, stderr = process.communicate()
 
     time_convert_end = time.time()
     time_process = time_convert_end - time_convert_start
@@ -741,8 +745,9 @@ def terminate_procmon(procmonexe):
 
     cmdline = '"{}" /Terminate'.format(procmonexe)
     log_debug('[*] Running cmdline: {}'.format(cmdline))
-    stdnull = subprocess.Popen(cmdline)
-    stdnull.wait()
+    process = subprocess.Popen(cmdline, stdout=subprocess.PIPE,
+                               stderr=subprocess.PIPE)
+    stdout, stderr = process.communicate()
 
 
 def parse_csv(csv_file, report, timeline):