Skip to content

Commit

Permalink
Add some documentation in the default rules.
Browse files Browse the repository at this point in the history
  • Loading branch information
jvoisin committed Nov 3, 2023
1 parent cb3d7ae commit 320b3f8
Showing 1 changed file with 1 addition and 2 deletions.
3 changes: 1 addition & 2 deletions config/default_php8.rules
Original file line number Diff line number Diff line change
Expand Up @@ -52,8 +52,7 @@ sp.disable_function.function("putenv").param("assignment").value_r("GCONV_").dro
sp.disable_function.function("extract").param("array").value_r("^_").drop()
sp.disable_function.function("extract").param("flags").value("0").drop()

# This is also burned:
# ini_set('open_basedir','..');chdir('..');…;chdir('..');ini_set('open_basedir','/');echo(file_get_contents('/etc/passwd'));
# See https://dustri.org/b/ini_set-based-open_basedir-bypass.html
# Since we have no way of matching on two parameters at the same time, we're
# blocking calls to open_basedir altogether: nobody is using it via ini_set anyway.
# Moreover, there are non-public bypasses that are also using this vector ;)
Expand Down

0 comments on commit 320b3f8

Please sign in to comment.