Suppress arginfo / zpp mismatch via ZEND_SUPPRESS_ARGINFO_ZPP_MISMATCH

[zeriyoshi]

Currently, debug builds of PHP always detect an arginfo / zpp mismatch in shared extensions and raise a Fatal Error while running extensions that do not properly define arginfo. This is very inconvenient if you want to get a core dump while running an extension (e.g. protobuf, redis) that does not define arginfo properly.

To solve this problem, the environment variable ZEND_SUPPRESS_ARGINFO_ZPP_MISMATCH has been added to disable arginfo / zpp integrity checks even in debug builds when it is enabled.

In PHP 8.2, a change was made to disable detection of Fake Closure arginfo / zpp mismatches, a side effect of which is that arginfo / zpp integrity checks are no longer performed for extensions built into PHP itself. It appears that this change is aready unnecessary, so we will revert it at the same time.

Update: This is bug and fixed in #10417

In any case, this change affects only debug builds and does not affect normal builds.

Closes: #10451

[nielsdos]

This seems kinda related to #10417, where I found that the check introduced in 8.2 wrongly disabled validation in that function by creating a condition which is always true.

[zeriyoshi]

@nielsdos
I am not familiar with Fake Closures, but apparently this PR solves the problem.
I'll rebase and push again when #10417 is merged :)

[zeriyoshi]

#10445

[Girgias]

@zeriyoshi is this still relevant with #10417 merged or not?

[zeriyoshi]

@Girgias
It has already been merged and rebased so there are no conflicts

I am only concerned if this change is acceptable as a patch version change. Since it only affects debug builds, I don't see it as a problem...

[nikic]

I am only concerned if this change is acceptable as a patch version change. Since it only affects debug builds, I don't see it as a problem...

The patch is ABI-breaking, so definitely not acceptable in this form.

[zeriyoshi]

@nikic
I see. Thank you very much.
I assume this change only affects debug builds and not the ABI of release builds, but should the ABI of debug builds also be guaranteed?

But I don't particularly want this change in PHP-8.2 branch, so I would like to make the same change for master. Is the ABI change acceptable in this case?

[iluuu1994]

@zeriyoshi The ABI break would definitely be acceptable for master.

[zeriyoshi]

@iluuu1994
If this PR is properly rebuilt for the master, is there any chance it could be incorporated? If so, I would be happy to resume work!

[iluuu1994]

@zeriyoshi Sure! Should be fine for master.

[zeriyoshi]

Thanks! I rework it.

[iluuu1994]

@zeriyoshi Would you still like to merge this? In that case, please rebase and target master.