Update fuzzer parser dict for property hooks words

[jorgsowa]

• adds two words to fuzzer parser dict coming from property hooks feature set and get
• removes redundant unset word

[nielsdos]

Thanks

[iluuu1994]

I don't know what this dictionary is needed for, but get and set are not keywords with the hooks rfc.

[nielsdos]

I don't know what this dictionary is needed for, but get and set are not keywords with the hooks rfc.

A fuzzer can mutate an input corpus randomly, but that's not very efficient to hit new code paths when testing certain features. So fuzzers are often given a dictionary of interesting keywords (well, they don't have to be _key_words, just syntactic elements suffices) that can hit new code paths.

[iluuu1994]

Do we then need to provide a new input corpus with hooks to effectively fuzz hooks?

[jorgsowa]

Thanks @nielsdos for the explanation. Parser fuzzer is used by OSS-fuzz, so it helps to generate more precise code paths for the master branch.

[jorgsowa]

Do we then need to provide a new input corpus with hooks to effectively fuzz hooks?

No, it gets the corpus automatically from the tests located in the Zend/tests directory.
https://github.com/php/php-src/blob/master/sapi/fuzzer/generate_parser_corpus.php

[nielsdos]

Do we then need to provide a new input corpus with hooks to effectively fuzz hooks?

What jorg says is true, it comes from phpt files. Which is also the reason that we have found variants of issues in the past shortly after the fix for an issue was committed.

[iluuu1994]

Great! Thanks for the explanation