Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Current command injection behavior isn't correct #35

Closed
ericwb opened this issue Apr 26, 2018 · 1 comment
Closed

Current command injection behavior isn't correct #35

ericwb opened this issue Apr 26, 2018 · 1 comment

Comments

@ericwb
Copy link
Member

ericwb commented Apr 26, 2018

Consider the following test file:

import subprocess
my_val = 'do_something; ' + evil_value
subprocess.Popen(my_val, shell=True)

This is obviously a high risk command injection issue, but Bandit currently only reports a low. Since all we can tell is that it's a dynamically constructed string we have to assume HIGH severity IMO.

@ericwb
Copy link
Member Author

ericwb commented Apr 26, 2018

Bad import of bug from launchpad. Closing..

@ericwb ericwb closed this as completed Apr 26, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant