Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fixed bug #79468
  NEWS

Merge branch 'PHP-7.2' into PHP-7.3

* PHP-7.2:
  Fixed bug #79468
  NEWS

Fixed bug #79468

Close the stream filter resources when removing them from the stream.

NEWS

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Don't leak peername if accept fails

Don't leak peername if accept fails

Even if the accept fails, the peername may be populated.

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix test cases

Fix test cases

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fixed bug #79468

Fixed bug #79468

Close the stream filter resources when removing them from the stream.

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Add NEWS entries [ci skip]

Add NEWS entries [ci skip]

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix bug #79465 - use unsigneds as indexes.
  Fix bug #79330 - make all execution modes consistent in rejecting \0

Merge branch 'PHP-7.2' into PHP-7.3

* PHP-7.2:
  Fix bug #79465 - use unsigneds as indexes.
  Fix bug #79330 - make all execution modes consistent in rejecting \0

Fix bug #79465 - use unsigneds as indexes.

Fix bug #79330 - make all execution modes consistent in rejecting \0

Fixed bug #78434

The DO_INIT flag, which will skip the first resume on a primed
generator, should always be set when starting to yield from a
new generator, not only when the yield from happens during priming.

Add php_cli_server_connect() helper

To encapsulate the repeated fsockopen() code.

This gives us a chance to control the timeout in one place:
Raise it to one second.

Make mysqli_poll test more deterministic

Handle errors appearing in different order.

Relax overly strict test expectation

There is no reason to expect a `1` after the PID; neither the session
ID nor the memory usage are required to contain one.  Actually, we just
want to verify here, that the process with the $child_pid is running,
and is a php.exe process.

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix memory leak introduced by fixing bug #78221

Fix memory leak introduced by fixing bug #78221

We have to free the retrieved text content; to keep the code readable,
we extract a helper function to check for empty nodes.  Unfortunately,
we cannot use xmlIsBlankNode(), because that also recognizes whitespace
only text content.

We also make sure to properly handle NULL returns from
xmlNodeGetContent().

Fix test

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Revert "Went to fast and forgot to update tests"
  Revert "Fix Bug #79448 0 is a valid Unicode codepoint, but mb_substitute_character(0) fails"

Revert "Went to fast and forgot to update tests"

This reverts commit 656eac74fa6074aebc087bb73d2e4651f7dc8c9e.

Revert "Fix Bug #79448 0 is a valid Unicode codepoint, but mb_substitute_character(0) fails"

This commit brings some substantial changes in behaviour due to the weird implementation.
This will be fixed in master due to BC concerns.

This reverts commit 1333b46d6dc0c293c1fd626803f91bc69743eb79.

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix #78221: DOMNode::normalize() doesn't remove empty text nodes

Fix #78221: DOMNode::normalize() doesn't remove empty text nodes

If a text node is not followed by another text node, we remove it, if
its textContent is empty.

Fix #69264: __debugInfo() ignored while extending SPL classes

We actually implement `::__debugInfo()` and drop the `get_debug_info()`
handlers of all relevant SPL classes.  This is cleaner and gives more
flexibility regarding overriding the functionality in descendant
classes.

Merge branch 'PHP-7.3' into PHP-7.4

Went to fast and forgot to update tests

However due to the really lax conversion to integer all strings pass as 0

Merge branch 'PHP-7.3' into PHP-7.4

Fix Bug #79448 0 is a valid Unicode codepoint, but mb_substitute_character(0) fails

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix incorrect free for last_message

Fix incorrect free for last_message

In commit a7305eb539596e175bd6c3ae9a20953358c5d677 the last_message
field of the connection object was changed to be always non-persistent.
But there is a place on change_user path that still treats it
depending on conn->persistent flag. This will cause PHP crash after
com_change_user success when there is last_message set

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fixed bug #79434

Fixed bug #79434

Skip fpm tests not designed to be run as root

When running as root with TEST_FPM_RUN_AS_ROOT=1

Allow fpm tests to be run with long socket path

Socket path is restricted to ~100 bytes, so we can use
the system temp dir if the path ends up too long.

Enable error diagnostic for check_default_conf_path.phpt

Otherwise we have no clue why the test failed, if the regex didn't
match.

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix literal compaction collision between string and double

Fix literal compaction collision between string and double

For the sake of simplicity I'm using a separate hashtable, rather
than trying to do hash perturabation on the double strings.

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Add NEWS entryc [ci skip]

Add NEWS entryc [ci skip]

Merge branch 'PHP-7.3' into PHP-7.4

Fix bug 79441

The 7.4 branch is now for 7.4.6

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Next is 7.3.18

Next is 7.3.18

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix #79413: session_create_id() fails for active sessions

Fix #79413: session_create_id() fails for active sessions

The comment on `PS_VALIDATE_SID_FUNC(files)` is very clear that the
function is supposed to return `SUCCESS` if the session already exists.
So to detect a collision, we have to check for `SUCCESS`, not
`FAILURE`.

We also fix the wrong condition in session_regenerate_id() as well.

Don't check directory nlink in stat tests

It seems like on many filesystems nlink for directories is the
number of subdirectories (plus two, due to . and ..). However,
this is not a POSIX requirement, and some filesystems don't
implement it this way. This seems to be the case for whatever is
used on the Travis AArch64 builders now.

Fix #74940: DateTimeZone loose comparison always true

Since `DateTimeZone` does not implement a `compare_objects` handler,
nor has any properties, two `DateTimeZone` instances always compare as
being equal, even if they designate totally different timezones.  Even
worse, after calling `var_dump()` on these objects, the actual
comparison may yield a correct result.

We therefore introduce a `compare_objects` handlers, which prevents
different behavior before/after `var_dump()`, and which allows us to
clearly define the intended semantics.

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix #79427: Integer Overflow in shmop_open()

Fix #79427: Integer Overflow in shmop_open()

If `shm.shm_segsz > ZEND_LONG_MAX` the assignment to `shmop->size` a
few lines below would overflow, so we catch that early and bail out if
necessary.

Allow numeric [UG]ID in FPM listen.{owner,group}

NEWS

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix #79424 ext/zip: don't use gl_pathc after call to globfree

Fix #79424 ext/zip: don't use gl_pathc after call to globfree

This breaks on Linux with the musl libc, since it zeroes out gl_pathc during
globfree.

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Add missing 'skip' to bug79332.phpt skip message

Add missing 'skip' to bug79332.phpt skip message

The skip message must start with the word 'skip', otherwise the test will not be skipped.

*Before:*
Running selected tests.
TEST 1/1 [ext/com_dotnet/tests/bug79332.phpt]
========DIFF========
001+ Fatal error: Uncaught Error: Class 'COMPersistHelper' not found in /srv/php/ext/com_dotnet/tests/bug79332.php:2
001- A com_exception has been thrown
002+ Stack trace:
003+ #0 {main}
004+   thrown in /srv/php/ext/com_dotnet/tests/bug79332.php on line 2
========DONE========
FAIL Bug #79332 (php_istreams are never freed) [ext/com_dotnet/tests/bug79332.phpt]

*After:*
Running selected tests.
SKIP Bug #79332 (php_istreams are never freed) [ext/com_dotnet/tests/bug79332.phpt] reason: com_dotnet extension not available

Explicitly declare the char as signed in zend_ffi_val.

This causes issues down the line as char are unsigned on some platforms,
e.g. ARM and cause a [-Wtype-limits] warning to be emitted.

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fixed bug #79412 (Opcache chokes and uses 100% CPU on specific script).

Fixed bug #79412 (Opcache chokes and uses 100% CPU on specific script).

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix bug #79410 (system() swallows last chunk if it is exactly 4095 bytes without newline)

Fix bug #79410 (system() swallows last chunk if it is exactly 4095 bytes without newline)

Closes GH-5292.

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Add test file

Add test file

Forgot the git add again...

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Handle NULL caller_call_opline

Handle NULL caller_call_opline

This can happen if there is an EXIT in the call arguments, in which
case the DO_CALL opcode may be eliminated as unreachable.

Fix leak on Windows as well

Cf. <http://git.php.net/?p=php-src.git;a=commit;h=db08ef0d3274b239a6b9e68d71d02bb6acb71d82>

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix test for Windows

Fix test for Windows

Windows filenames may very well contain a colon, so we adjust the test
accordingly.

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix #79200: Some iconv functions cut Windows-1258

Fix #79200: Some iconv functions cut Windows-1258

To cater to potentially state-dependent encodings, we have to reset the
conversion descriptor into its initial shift state to properly finish
the conversion.  Furthermore, state-dependent encodings may not show
progress when comparing `in_left` before and after the conversion; we
rather have to see whether `out_left` has decreased.  Also we have to
cater to the fact that the final potentially state resetting call does
not signal failure, but we still have to break respective loops
afterwards.

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix #79393: Null coalescing operator failing with SplFixedArray

Fix #79393: Null coalescing operator failing with SplFixedArray

We favor the KISS principle over optimization[1] – SPL is already
special enough.

[1] <https://github.com/php/php-src/pull/2489/commits/352f3d4476a79bb86136b431719df7394e5a8d4e#r112498098>ff

Fix RSA memory leak in mysqlnd auth

unneeded after fix

NEWS

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  NEWS
  Fix Bug #79296 ZipArchive::open fails on empty file

NEWS

Fix Bug #79296 ZipArchive::open fails on empty file

Mention which categories to use for patch releases

Fix potential test conflict

Cf. <https://ci.appveyor.com/project/php/php-src/builds/31564684/job/k9u3xv860fao3n2e#L5578>

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix #79396: DateTime hour incorrect during DST jump forward

Fix #79396: DateTime hour incorrect during DST jump forward

When you attempt to set the time to a non-existent time occuring during
a DST jump forward, the hour does not move forward correctly.

Improve FFI test suite for Windows

We add Windows support to four existing test cases, extract some useful
utility functions, and use them to simplify further test cases.

We also remove the Windows specific code from preload.inc, since
preloading isn't supported on Windows anyway.

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Clarify session.cookie_samesite="None"

Clarify session.cookie_samesite="None"

Make bug52820.phpt more robust

Use a more robust error condition...

Explicitly start mysql

Use "set -e" in some pipeline steps

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  bump verison to 7.2.30-dev

Merge branch 'PHP-7.2' into PHP-7.3

* PHP-7.2:
  bump verison to 7.2.30-dev

bump verison to 7.2.30-dev

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix NEWS

Merge branch 'PHP-7.2' into PHP-7.3

* PHP-7.2:
  Fix NEWS

Fix NEWS

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  [ci skip] Update NEWS
  Fix test
  Fix bug #79329 - get_headers should not accept \0
  Fixed bug #79282