-
-
Notifications
You must be signed in to change notification settings - Fork 422
feat: update unix commands list #4215
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: update unix commands list #4215
Conversation
|
📊 Quantitative test results for language: |
|
What do you mean by "broken"? And yes. The RCE rules are a pain. |
the
I'll open an issue on this so we can get some movement on this |
Can you give me an example? Apparently, |
|
It doesn't, see the output below:
|
|
@theseion I noticed the script in |
|
Thanks. Let me look at some stuff :) |
|
932250 is written in such a way that the modifiers are ignored. It will only append an approximation of the |
|
The issue with 932250 should be fixed with #4231. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I assume this is for plain pip, as opposed to pip3. Please add a test.
|
@theseion Done Is |
For URLs only. Data is sent verbatim. |
tests/regression/tests/REQUEST-932-APPLICATION-ATTACK-RCE/932250.yaml
Outdated
Show resolved
Hide resolved
Co-authored-by: Max Leske <250711+theseion@users.noreply.github.com>
for more information, see https://pre-commit.ci
Adds some commands for AppArmor, bubblewrap, auditd,
ping6,pipand capabilities commands.I couldn't get the AppArmor commands to be blocked at PL-1 because it uses the
~operator to detect permutations and that operator is broken with some rules. The Unix rules are hard to work with, evasion operators are defined inconsistently and all over the place, and there seems to be some overlapping rules too.