Skip to content

Commit

Permalink
Fix yet another php surprised-rename of functions parameters
Browse files Browse the repository at this point in the history
  • Loading branch information
jvoisin committed Mar 24, 2024
1 parent 435977a commit 6fe3815
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion config/default_php8.rules
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,9 @@ sp.disable_function.function("chmod").param("permissions").value("438").drop();
sp.disable_function.function("chmod").param("permissions").value("511").drop();

# Prevent various `mail`-related vulnerabilities
sp.disable_function.function("mail").param("additional_parameters").value_r("\\-").drop();
# Uncommend the second rule if you're using php8.3+
#sp.disable_function.function("mail").param("additional_parameters").value_r("\\-").drop();
#sp.disable_function.function("mail").param("additional_params").value_r("\\-").drop();

# Since it's now burned, me might as well mitigate it publicly
sp.disable_function.function("putenv").param("assignment").value_r("LD_").drop()
Expand Down

0 comments on commit 6fe3815

Please sign in to comment.