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

Signatures for the Symfony Process component #37

Open
chrisdeeming opened this issue Oct 8, 2021 · 0 comments
Open

Signatures for the Symfony Process component #37

chrisdeeming opened this issue Oct 8, 2021 · 0 comments
Labels
enhancement New feature or request

Comments

@chrisdeeming
Copy link

The Antimalware Scanner would detect the existence of the Symfony Process component as it makes extensive use of the exec proc_* and posix functions but one thing the scanner doesn't account for is if the environment - by design - already includes that component but the code we're scanning may be attempting to use it which may have undesirable effects.

"Don't have the Symfony Process component in your environment" seems like a sensible solution but one that doesn't work for us unfortunately.

For example, this code would be blocked by the scanner:

exec("rm -rf /path/to/delete");

But this code wouldn't be:

$process = new Process("rm -rf /path/to/delete");

Yet the effect is the same.

Is it worth adjusting the scanner to detect potential usages of this component? There may be others out there, of course, but Symfony Process is the only one that affects us at this time.

@marcocesarato marcocesarato added the enhancement New feature or request label Oct 13, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants