Fix Windows build

Try to fix windows build

Add Z_PARAM_ITERABLE and co

Refactor parts of SPL Dir/SplFileObject

This fixes a way it was possible to trigger an Internel Error
by disabling function (via the INI setting) when SPL was acting
as a proxy to the function call.

Fix flock_compat layer as it needs to used in SPL now.

Use macro to check if object is initialized

Closes GH-6014

Fix mismatch between macro and struct definition

Fix leaks in sapi tests

Make sure to always free compiled_filename on shutdown.

Don't intern compiled_filename

For php-ast interning the file name is an effective memory leak,
see php-ast#134.

I don't think there's any reason to do this. At some point this
was needed due to bugs in the interned string mechanism that
caused issues if the string was later interned, e.g. through a
__FILE__ reference. These issues have since been resolved.

In conjunction with the filenames_table removal in c4016ecd446ef26bb3dc77735b6e441e151ea985
this means that filenames now need to be refcounted like normal
strings. In particular the filename reference in op_arrays and CEs
are refcounted.

Remove CG(filenames_table)

This doesn't seem to serve any purpose anymore.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fixed bug #80046

Fixed bug #80046

We already protect against optimizing away loop frees in DFA pass,
but not in block pass.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fixed bug #80046

Fixed bug #80046

We already protect against optimizing away loop frees in DFA pass,
but not in block pass.

JIT for FE_FETCH_R

Merge branch 'PHP-7.4' into master

* PHP-7.4:
  Skip test if A: drive exists

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Skip test if A: drive exists

Skip test if A: drive exists

Otherwise the test case will fail for a very different reason.

Fixed bug #80045

Applying the obvious fix ... however, I think we may need to
rething how we handle trampoline fcc for "f" zpp. It might make
sense to use fcc->function_handler == NULL for that case and
force it to be fetched in zend_call_function instead (it will
be reset to that after the call anyway). Otherwise we will keep
chasing these leaks, as it's the only instance where it's
necessary to free a zpp result.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Handle memory limit error during string reallocation correctly

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Handle memory limit error during string reallocation correctly

Fixed support for deprecated constants (Zend/tests/const_deprecation.phpt failure)

Handle memory limit error during string reallocation correctly

Do not decrement the refcount before allocating the new string,
as the allocation operation may bail out and cause a use-after-free
lateron. We can only decrement the refcount once the allocation
has succeeded.

Fixes oss-fuzz #25384.

JIT for FETCH_CONSTANT

Reorder switch cases for consistency between zend_jit() and zend_jit_trace()

libmagic: Update patch

[ci skip]

Signed-off-by: Anatol Belski <ab@php.net>

Fix typos in mbstring tests

Man, I can be pedantic sometimes. Tiny little things like misspelled words just
hurt me inside. So while it's not really a big deal, I couldn't leave these typos
alone...

Update NEWS, UPGRADING

Change Attribute Syntax from @@ to #[]

Revert "libmagic: Move the allocation on the stack"

This reverts commit 1d84a58736541b61fc9d569580715940e32d55c9.

Signed-off-by: Anatol Belski <ab@php.net>

JIT for FE_FETCH_R

libmagic: Constify arg

libmagic: Move the allocation on the stack

hash: Fix warning in the bench script

Extend function blacklist in execute fuzzer

Add pfsockopen and stream_socket_server.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fix binary-safety of parse_url

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix binary-safety of parse_url

Fix binary-safety of parse_url

php_parse_url() is intended to support strings that are not zero
terminated. We can't use strcspn in the implementation.

As we have two uses of strcspn, add a helper.

Micro-optimization

X86: Fast CRC32 computation using PCLMULQDQ instruction

Based on:
"Fast CRC Computation for Generic Polynomials Using PCLMULQDQ Instruction"
V. Gopal, E. Ozturk, et al., 2009, http://intel.ly/2ySEwL0

Signed-off-by: Frank Du <frank.du@intel.com>
Closes GH-6018

Fix typo in test description

Closes GH-6062.

Added missing helper

Improved JIT for VERIFY_RETURN_TYPE

Move custom type checks to ZPP

Closes GH-6034

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fix throwing of yield from related exceptions into generator

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix throwing of yield from related exceptions into generator

Fix throwing of yield from related exceptions into generator

Use the general zend_generator_throw_exception() helper for this.
Otherwise we don't handle the off-by-one opline correctly (should
we maybe just stop doing that?)

This is a followup to ad750c3bb6e7b48384c6265eb9d3bcf5b4000652,
which fixed a different yield from exception handling problem that
happened to show up in the same test case from oss-fuzz #25321.
Now both issues should be fixed.

Minimal JIT support for JMP_NULL

Fix by-ref list assign LIST_W+MAKE_REF separation

Shift the responsibility for emitting MAKE_REF to the list assignment
code, to make sure that LIST_W and MAKE_REF are directly adjacent,
and there are no opcodes in between that could modify the LIST_W
result.

Additionally, adjust the zend_wrong_string_offset() code to not
perform a loop over opcodes and assert that the next opcode is
a relevant one. The VM write-safety model requires this.

This is a followup to a07c1f56aac1c0f6c8334760009b678cbf9d6138
and the full fix for oss-fuzz #25352.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fix infinite loop on string offset during by-ref list assign

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix infinite loop on string offset during by-ref list assign

Fix infinite loop on string offset during by-ref list assign

There is a deeper underlying issue here, in that the opcodes violate
VM write-fetch safety, but let's fix the infinite loop first.

This fixes oss-fuzz #25352.

Preallocate zval for unused result of internal function on CPU stack

Disable ifunc resolvers under dataflow sanitizer

As with other sanitizers, this is not supported.

Assert there are children in zend_generator_get_child()

Update NEWS for 8.0.0rc1

Update NEWS for PHP 8.0.0beta3

Add zend_observer API

Closes GH-5857.

Co-authored-by: Nikita Popov <nikita.ppv@gmail.com>
Co-authored-by: Sammy Powers <sammyk@datadoghq.com>

Eliminate useless EG(exception) checks aftr FE_RESET/FE_FETCH

Add more precise type info for stubs

Closes GH-6005

Eliminate some EX_CALL_INFO() checks

Merge branch 'PHP-7.4'

* PHP-7.4:
  Check for null EX(func) in write_property

Check for null EX(func) in write_property

This can happen if zend_call_function inserted a dummy frame,
and we already switched to the dummy frame in leave_helper,
and an exception is thrown during CV destruction.

Fixes oss-fuzz #25343.

Prepend compiler warning flags to CFLAG instead to append them

Credit to @bwoebi for the idea

Closes GH-6057

Fix FE_FETCH_R_SIMPLE specialization

After a99d08b5d135fffa1d83b08b056e0080d295d863 the type can
include UNDEF. However, UNDEF can only reach FE_FREE, not FE_FETCH.
As such, simply ignore this type.

Avoid null pointer UB in accel blacklist

Pretty annoying...

Promote warnings to exceptions in ext/soap and ext/xmlwriter

Closes GH-5998

Promote warnings to exceptions in ext/tidy

Closes GH-6051

Fix type inference for FE_RESET on invalid type

FE_RESET sets the result to UNDEF in this case. We should infer
some type here, because no type generally implies unreachable
code. In this case SCCP ended up replacing the FE_RESET result
with null, including in FE_FREE.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fix pi node removal when removing predecessor

Add explicit double cast

Avoid precision warning.

Fix pi node removal when removing predecessor

We can't just remove the uses, we need to replace uses.

The test case only fails on master with SSA integrity violations,
but I believe the root issue also existed previously.

Fixed nested init fcall guards.

Code cleanup in mbstring.c

Comment constants in mbfl_consts.h, remove unused ones

These were unused, and almost certainly will never be used:

- MBFL_ENCTYPE_MWC4BE
- MBFL_ENCTYPE_MWC4LE
- MBFL_ENCTYPE_SHFTCODE
- MBFL_ENCTYPE_ENC_STRM

For the latter two, there were some encodings which were marked with these flags;
but nothing ever _checked_ these particular flags.

Add comment on mbstring East Asian Width table

Don't guard mbstring code with #ifdef HAVE_MBSTRING

This is just a very silly feature of mbstring -- you can compile the source files with
HAVE_MBSTRING undefined, and it will all just compile to (almost) nothing. What is the
use of this? Why compile the source files and link against them if you don't want the
mbstring extension? It doesn't make any kind of sense.

Remove redundant includes from mbstring (and make sure correct config.h is used)

Very interesting... it turns out that when Valgrind support was enabled,
`#include "config.h"` from within mbstring was actually including the file "config.h"
from Valgrind, and not the one from mbstring!!

This is because -I/usr/include/valgrind was added to the compiler invocation _before_
-Iext/mbstring/libmbfl.

Make sure we actually include the file which was intended.

Remove useless definition of NULL in mbfl_string.h

If NULL is not defined by the platform, mbfl_defs.h already defines it.

Optimize php_unicode_convert_case (cuts mbstring case conversion time ~15%)

This function uses various subfunctions to convert case of Unicode wchars.
Previously, these subfunctions would store the case-converted characters in
a buffer, and the parent function would then pass them (byte by byte) to
the next filter in the filter chain.

Rather than passing around that buffer, it's better for the subfunctions to
directly pass the case-converted bytes to the next filter in the filter chain.
This speeds things up nicely.

Fix typos in comments in mb_send_mail

Remove unused functions from mbstring

- mbfl_buffer_converter_reset
- mbfl_buffer_converter_strncat
- mbfl_buffer_converter_getbuffer
- mbfl_oddlen
- mbfl_filter_output_pipe_flush
- mbfl_memory_device_output2
- mbfl_memory_device_output4
- mbfl_is_support_encoding
- mbfl_buffer_converter_feed2
- _php_mb_regex_globals_dtor
- mime_header_encoder_feed
- mime_header_decoder_feed
- mbfl_convert_filter_feed

Separate implementation of mb_{en,de}code_numericentity

Rather than using a magic boolean parameter to choose different behavior of
the subfunction, inline it. The code size doesn't really grow anyways. And
soon these will be trimmed down more.

Use new-style argument parsing macros in mbstring.c

Inline unneeded indirection for mbstring memory management

All memory allocation and deallocation for mbstring bounces through a table of
function pointers before going to emalloc/efree/etc. But this is unnecessary.
The allocators are never swapped out. Better to just call them directly.

Additional tests for mbstring extension

Prevent com::__construct() to modify the $server_name array

We switch to `zend_string`s for simplicity, so there's no need to
`convert_to_string()` anymore, what makes the array separation
superfluous.

Closes GH-6040

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fix leak on assign concat of array and empty string

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix leak on assign concat of array and empty string

Fix leak on assign concat of array and empty string

Fix nullsafe operator with delayed oplines

Closes GH-6056.

Add tests for nullsafe operator on delayed oplines

More agressive "packed array" guards insertion

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fix leak on consteval exception in ReflectionClass::__toString()

Fix leak on consteval exception in ReflectionClass::__toString()

Merge branch 'PHP-7.4'

* PHP-7.4:
  Check update constant failure in ReflectionClassConstant::__toString()

Check update constant failure in ReflectionClassConstant::__toString()

Fix build warnings after timelib update

Perform type guard checks before IS_UNDEF checks (check IS_UNDEF during deoptimization)

Fix fetching default value of internal function with userland arginfo

"Fix" in the sense of "not crash". We aren't able to actually
display the default value for this case, as there's no way to
fetch the relevant information right now.

Merge branch 'PHP-7.4' into master

Merge branch 'PHP-7.3' into PHP-7.4