Skip to content
This repository was archived by the owner on Feb 27, 2020. It is now read-only.

Fixed incorrectly identifying get_magic_quotes_runtime as deprecated #52

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

mheesters
Copy link

I'm getting results like these:

/var/www/something/FileByteStream.php
deprecatedFunctions
Line 42: if (function_exists('get_magic_quotes_runtime') && @get_magic_quotes_runtime() == 1) {

It looks like the script falsely identifies get_magic_quotes_runtime as being a deprecated function.

It's not: https://www.php.net/manual/en/function.get-magic-quotes-runtime.php whereare magic_quotes_runtime is indeed deprecated: https://www.php.net/manual/en/function.magic-quotes-runtime.php

I've extented the regex so that the target shouldn't be preceded by a underscore.

I'm getting results like these:

```
/var/www/something/FileByteStream.php
deprecatedFunctions
Line 42: if (function_exists('get_magic_quotes_runtime') && @get_magic_quotes_runtime() == 1) {
```

It looks like the script falsely identifies `get_magic_quotes_runtime` as being a deprecated function.

It's not: https://www.php.net/manual/en/function.get-magic-quotes-runtime.php whereare magic_quotes_runtime is indeed deprecated: https://www.php.net/manual/en/function.magic-quotes-runtime.php

I've extented the regex so that the target shouldn't be preceded by a underscore.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant