Do not run omc with shell=True

Author: ondras12345

OMPython/__init__.py

@@ -188,7 +188,7 @@ def _start_omc_process(self, timeout):
             my_env = os.environ.copy()
             my_env["USER"] = self._currentUser
             # Because we spawned a shell, and we need to be able to kill OMC, create a new process group for this
-            self._omc_process = subprocess.Popen(self._omc_command, shell=True, stdout=self._omc_log_file, stderr=self._omc_log_file, env=my_env, preexec_fn=os.setsid)
+            self._omc_process = subprocess.Popen(self._omc_command, stdout=self._omc_log_file, stderr=self._omc_log_file, env=my_env, preexec_fn=os.setsid)
         if self._docker:
             for i in range(0, 40):
                 try:
@@ -276,7 +276,7 @@ def _set_omc_command(self, omc_path_and_args_list):
         if sys.platform == 'win32':
             self._omc_command = omc_path_and_args_list
         else:
-            self._omc_command = ' '.join([shlex.quote(a) for a in omc_path_and_args_list])
+            self._omc_command = [shlex.quote(a) for a in omc_path_and_args_list]
 
         return self._omc_command