There's rather a lot of subprocess.Popen usage here, many of which use string formatting of one form or another to build the command they run. This may or not be insecure, we just don't know. Using the list based approach would be better.
Likely other utils in subprocess can also now simplify them.
There's rather a lot of
subprocess.Popen
usage here, many of which use string formatting of one form or another to build the command they run. This may or not be insecure, we just don't know. Using the list based approach would be better. Likely other utils insubprocess
can also now simplify them.