Merge branch 'PHP-7.4'

* PHP-7.4:
  Fix freeing of dynamic call name

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix freeing of dynamic call name

Fix freeing of dynamic call name

We need to free op2 if the call construction fails.

Also remove a redundant check for !call.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fix const/cv freeing on failed reference assignment

Fix const/cv freeing on failed reference assignment

Merge branch 'PHP-7.4'

* PHP-7.4:
  More updates to CONTRIBUTING.md

More updates to CONTRIBUTING.md

7.4 is stable.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Update CONTRIBUTING.md

Update CONTRIBUTING.md

7.1 is EOL; 7.2 is security fixes only.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fix #78976: SplFileObject::fputcsv returns -1 on failure

Fix #78976: SplFileObject::fputcsv returns -1 on failure

`php_stream_write()` returns `-1` on failure now, which has to be
catered to by `SplFileObject::fputcsv()` which still is supposed to
return `false` on failure.

Fix leak of dynamic property name in address helper

Merge branch 'PHP-7.4'

* PHP-7.4:
  Free buffer in php_verror even if it is empty

Free buffer in php_verror even if it is empty

vspprintf allocates a buffer even if the resulting string is empty.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fix resetting of internal_encoding and friends

Fix resetting of internal_encoding and friends

We need to update the value even if new_value is NULL. In particular,
it should be reset back to NULL after each request if the setting was
not specified on startup. Otherwise we leave dangling pointers.

Avoid generation of "jmp" to next instruction

Merge branch 'PHP-7.4'

* PHP-7.4:
  Avoid float to int cast UB in exif

Avoid float to int cast UB in exif

Make "exception_handler_undef" stub to check EG(opline_before_exception) and undefine corresponnding result variable, if necessary.
This fixes few invalid "exception_handler_undef" usages.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Update/fix NEWS [ci skip]

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Update/fix NEWS [ci skip]

Update/fix NEWS [ci skip]

Merge branch 'PHP-7.4'

* PHP-7.4:
  [ci skip] Update NEWS
  [ci skip] Update  NEWS
  Fix test

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  [ci skip] Update NEWS
  [ci skip] Update  NEWS
  Fix test

Merge branch 'PHP-7.2' into PHP-7.3

* PHP-7.2:
  [ci skip] Update  NEWS

[ci skip] Update NEWS

[ci skip] Update  NEWS

Merge branch 'PHP-7.4'

* PHP-7.4:
  Property names of internal classes from temporary extensins, loaded by dl(), may be emalloc-ed strings.

Property names of internal classes from temporary extensins, loaded by dl(), may be emalloc-ed strings.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fixed memory leak

Fixed memory leak

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fixed memory leaks

Fixed memory leaks

Merge branch 'PHP-7.4'

* PHP-7.4:
  Set opcache.optimization_level=0 in test

Set opcache.optimization_level=0 in test

Otherwise opcache "inlines" the function, which causes a minor
difference in destruction behavior.

Fix tests

Fix test

Merge branch 'PHP-7.4'

* PHP-7.4:
  Test fixes

Test fixes

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fix test
  Fix bug #78793
  Fix build - no model field anymore
  Fixed bug #78910
  Fix #78878: Buffer underflow in bc_shift_addsub
  Fix test
  Fix #78862: link() silently truncates after a null byte on Windows
  Fix #78863: DirectoryIterator class silently truncates after a null byte
  Fix #78943: mail() may release string with refcount==1 twice

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix test
  Fix bug #78793

Merge branch 'PHP-7.4' of git.php.net:php-src into PHP-7.4

* 'PHP-7.4' of git.php.net:php-src:
  Remove DateTime class registration test
  Fixed bug #78973
  Fixed bug #78961 (erroneous optimization of re-assigned $GLOBALS)

Merge branch 'PHP-7.2' into PHP-7.3

* PHP-7.2:
  Fix test
  Fix bug #78793

Fix test

Merge branch 'PHP-7.4'

* PHP-7.4:
  Remove DateTime class registration test

Remove DateTime class registration test

This test is more annoying than useful.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fixed bug #78973

Fixed bug #78973

Save opline in leave helper to correctly handle destructor calls
during CV freeing (or other leave freeing).

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fixed bug #78961 (erroneous optimization of re-assigned $GLOBALS)

Fixed bug #78961 (erroneous optimization of re-assigned $GLOBALS)

Fix bug #78793

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix build - no model field anymore

Fix build - no model field anymore

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fixed bug #78910
  Fix #78878: Buffer underflow in bc_shift_addsub
  Fix test
  Fix #78862: link() silently truncates after a null byte on Windows
  Fix #78863: DirectoryIterator class silently truncates after a null byte
  Fix #78943: mail() may release string with refcount==1 twice

Merge branch 'PHP-7.2' into PHP-7.3

* PHP-7.2:
  Fixed bug #78910
  Fix #78878: Buffer underflow in bc_shift_addsub
  Fix test
  Fix #78862: link() silently truncates after a null byte on Windows
  Fix #78863: DirectoryIterator class silently truncates after a null byte

Fixed bug #78910

Fix #78878: Buffer underflow in bc_shift_addsub

We must not rely on `isdigit()` to detect digits, since we only support
decimal ASCII digits in the following processing.

Fix test

Fix #78862: link() silently truncates after a null byte on Windows

Since link() is supposed to accepts paths (i.e. strings without NUL
bytes), we must not accept arbitrary strings.

Fix #78863: DirectoryIterator class silently truncates after a null byte

Since the constructor of DirectoryIterator and friends is supposed to
accepts paths (i.e. strings without NUL bytes), we must not accept
arbitrary strings.

Fix #78943: mail() may release string with refcount==1 twice

Since we need `headers_lc` as well as `headers_trim` in the following,
we do not release the former even if they are the same string, to avoid
complicating the release logic even more.

A new test case is not necessary, since we already have
mail_basic_alt2-win32.phpt and others.

Further reduce parse depth limit in mbstring fuzzer

Depth limit of 1024 still causes optimize_node stack overflows
when running under asan.

Remove deprecated behaviour of passing encoding as third parameter in mb_strrpos()

Merged GH-5011

Revert "Remove dead code in libmbfl, memory device"

Stop trusting CLion's automatic dead code detection because it seems to be wrong
more often than not.
This reverts commit 612c86db3ea139e4ed4aa6a9aa28da343c9b39b8.

Remove dead code in sanitizing_filter.c of the filter extension

Remove dead code in libmbfl, memory device

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fixed bug #78921

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fixed bug #78921

Fixed bug #78921

By resetting fake_scope during autoloading. We already do the same
when executing destructors.

Limit parse depth in mbstring fuzzer

The default depth of 4096 is large enough to cause optimize_node
stack overflows under asan. Reduce to 1024.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Introduce extra counter to avoid RTD key collisions

Introduce extra counter to avoid RTD key collisions

Also generate a fatal error if a collision occurs in zend_compile.

This is not perfect, because collisions might still be introduced
via opcache, if one file is included multiple times during a request,
invalidate in the meantime and recompiled by different processes.

This still needs to be addressed, but this patch fixes the much
more common case of collisions occuring when opcache is not used.

Fixes bug #78903.

phar: fix offset*() prototypes

Fix exception message of implode()

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fix #78929: plus signs in cookie values are converted to spaces

Fix #78929: plus signs in cookie values are converted to spaces

We switch the cookie value parsing function from `php_url_decode()` to
`php_raw_url_decode()`, so that cookie values are now parsed according
to RFC 6265, section 4.1.1.  We also refactor to remove duplicate code
without changing the execution flow.

Remove support for mixing parameter order in implode()

Promote warnings to exceptions in password_*() functions

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fixed bug #78950: Preloading trait method with static variables

Fixed bug #78950: Preloading trait method with static variables

We need to make sure that trait methods with static variables
allocate a separate MAP slot for the static variables pointer,
rather than working in-place.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fix handling of non-final loop var free in sccp

Fix handling of non-final loop var free in sccp

We only need to preserve the FE_FREE that marks the end of the
loop range. Skip FE_FREEs with the FREE_ON_RETURN flag.

Add funcinfo for spl global functions

This assumes that `iterator_*` will now always throw or abort on failure.

Also, move #include _arginfo.h directive to the top of the file - virtually all
other files put it there, and developers may base code on basic_functions.c.

Closes GH-4968

Return empty string instead of NULL in serialize().

Modifiy its return type accordingly and arginfo.

Merge branch 'PHP-7.4'

* PHP-7.4:
  uint/ulong removal may affect other platforms as well

uint/ulong removal may affect other platforms as well

Merge branch 'PHP-7.4'

* PHP-7.4:
  Typo fix in php.ini-production

Typo fix in php.ini-production

Possibe -> Possible

Add stubs for standard library

Promote warnings to exceptions in sapi_windows_* functions

Promote warning to exception in unserialize()

Promote warning to exception in parse_url()

Promote warnings to exceptions for some filter functions

Merge branch 'array_filter-optimize' into HEAD

Merge branch 'PHP-7.4'

* PHP-7.4:
  Revert "Fixed bug #78903: Conflict in RTD key for closures results in crash"

Revert "Fixed bug #78903: Conflict in RTD key for closures results in crash"

This reverts commit b55033fa18afd9bb60fc4537270266d808ec0868.

This breaks ext/opcache/tests/bug65915.phpt.

Convert warnings to TypeError in php_spl.c

Closes GH-4991

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fixed bug #78903: Conflict in RTD key for closures results in crash

Fixed bug #78903: Conflict in RTD key for closures results in crash

I wasn't able to create a simple reproducer for this. General approach
is the same as for anonymous classes: If the key is already used, reuse
the old definition.