# Adding special case for "eval" in disable_functions INI setting

- Thread: https://externals.io/message/105484
- Original archive: https://news-web.php.net/php.internals/105484
- Started by Benjamin Eberlei on 2019-04-28T15:46:06+00:00
- Messages: 1
- Reply by email to internals@lists.php.net
- More for agents: search with https://externals.io/search.md?q={query} · JSON API at https://externals.io/api · documentation at https://externals.io/llms.txt

---

## #105484 — Benjamin Eberlei — 2019-04-28T15:46:06+00:00

URL: https://externals.io/message/105484#105484

Hi everyone,

I have added a patch to introduce a special case for eval when listed in
disable_functions INi setting. From a developer perspective it shouldn't be
necessary to know that eval is not a function but a language construct.
This is why I think this special case is needed rather than adding a new
INI setting.

https://github.com/php/php-src/pull/4084

The patch fixes https://bugs.php.net/bug.php?id=62397

It currently targets 7.4 branch, but the way its implemented I think it
could be merged into the stable branches.

Should I target the PR for 7.1/7.2 or where do the PMs think this should go?

greetings
Benjamin