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

feat(python): os command injection (CWE-78) #398

Merged
merged 3 commits into from
May 16, 2024

Conversation

elsapet
Copy link
Collaborator

@elsapet elsapet commented May 10, 2024

Description

Add Python rule for CWE-78 OS command injection

Checklist

If this is your first time contributing please sign the CLA

  • My rule has adequate metadata to explain its use.

@elsapet elsapet force-pushed the feat/python/add-common-user-input branch from 94bbf03 to 195151a Compare May 10, 2024 15:17
@elsapet elsapet force-pushed the feat/python/cwe-78-os-command-injection branch from a90e65f to 0b92cb1 Compare May 10, 2024 15:18
@elsapet elsapet force-pushed the feat/python/cwe-78-os-command-injection branch from 0b92cb1 to f43aff5 Compare May 13, 2024 07:45
@elsapet elsapet force-pushed the feat/python/add-common-user-input branch from f52cb55 to 9daae03 Compare May 13, 2024 14:39
Base automatically changed from feat/python/add-common-user-input to main May 14, 2024 15:18
@elsapet elsapet force-pushed the feat/python/cwe-78-os-command-injection branch from f43aff5 to fd99be4 Compare May 14, 2024 15:28
@elsapet elsapet marked this pull request as ready for review May 14, 2024 15:33
@elsapet elsapet requested a review from didroe May 14, 2024 15:34
@elsapet elsapet force-pushed the feat/python/cwe-78-os-command-injection branch from f3435f0 to 51e8e4e Compare May 15, 2024 13:09
@elsapet elsapet requested a review from didroe May 15, 2024 14:06
@elsapet elsapet merged commit 31e12b2 into main May 16, 2024
23 checks passed
@elsapet elsapet deleted the feat/python/cwe-78-os-command-injection branch May 16, 2024 09:44
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.

2 participants