fix

scripts/jo_utils.py

@@ -39,9 +39,8 @@ def _read_param_from_jo(jo, names, ignore_case=False):
 # See https://gitlab.cern.ch/atlas-physics/pmg/mcjoboptions/-/issues/98
 def _read_param_from_jo_withDict(jo, type, param):
     locals = {type: argparse.Namespace()}
-    skipLines=["os.system", "subprocess.call", "subprocess.Popen"] # skip lines containing these calls
     for line in jo.splitlines():
-        if any(x in line for x in skipLines): continue
+        if "os.system" or "subprocess.call" or "subprocess.Popen" in line: continue # for security
         try:
             exec(line, {}, locals)
         except:
@@ -56,11 +55,9 @@ def _env_set(jo, param):
     os.environ.clear()
     # dictionary to store environment variables
     locals = dict()
-    # skip lines containing these calls
-    skipLines=["os.system", "subprocess.call", "subprocess.Popen"]
     # Execute jO
     for line in jo.splitlines():
-        if any(x in line for x in skipLines): continue
+        if "os.system" or "subprocess.call" or "subprocess.Popen" in line: continue # for security
         try:
             exec(line.strip(), globals(), locals)
         except: