Skip to content

Fix OS command injection in database backup/restore utilities#3

Open
sheyabernstein wants to merge 1 commit intomainfrom
fix/db-command-injection
Open

Fix OS command injection in database backup/restore utilities#3
sheyabernstein wants to merge 1 commit intomainfrom
fix/db-command-injection

Conversation

@sheyabernstein
Copy link

  • Remove shell=True from subprocess calls
  • Use argument lists instead of shell string concatenation
  • Handle file I/O explicitly instead of shell redirection

Resolves CWE-78 (OS Command Injection) - CVSS 9.8

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

Successfully merging this pull request may close these issues.

1 participant