Merge branch 'PHP-7.4' into master

* PHP-7.4:
  [ci skip] Fix 7.3.21 release date

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  [ci skip] Fix 7.3.21 release date

[ci skip] Fix 7.3.21 release date

Fix opcache JIT on NetBSD with PaX

When PaX mprotect active, opcache JIT test fail as it needs
the PROT_MPROTECT macro to be able to allow further permission
promotion with mprotect.

Closes GH-5929.

XFAIL status-listen FPM test

Has frequent intermittent failures with:

ERROR: The NOTICE does not match expected message:
- PATTERN: /^\[\d\d-\w\w\w-\d{4} \d\d:\d\d:\d\d\] NOTICE: Terminating ...$/
- MESSAGE: [04-Aug-2020 08:15:37] WARNING: [pool unconfined_status] server reached max_children setting (2), consider raising it
- EXPECT: 'Terminating ...'
- ACTUAL: '2020 08:15:37] WARNING: [pool unconfined_status] server reached max_children setting (2), consider raising it'

Merge branch 'PHP-7.4'

* PHP-7.4:
  Manually build re2c on macos

Manually build re2c on macos

Avoid issues with the licensing mafia, see:
https://github.com/Homebrew/homebrew-core/pull/59094

Closes GH-5932.

Add opcache.jit=tracing|function values, make on/yes/true synonym for tracing.

Merge branch 'PHP-7.4'

* PHP-7.4:
  bump version to 7.2.34-dev
  Update NEWS

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  bump version to 7.2.34-dev
  Update NEWS

Merge branch 'PHP-7.2' into PHP-7.3

* PHP-7.2:
  bump version to 7.2.34-dev
  Update NEWS

bump version to 7.2.34-dev

Fix Socket related tests on Windows

Cleanup snmp after resource to object migration

Fix posix test

Fix Windows build

Add upgrading note for the resource to object migration in ext/sockets

Minor grammar/consistency fixes in upgrade guide

[ci skip]

Convert resources to objects in ext/sockets

Closes GH-5900.

Deprecate libxml_disable_entity_loader()

This method was used to protect code against XXE processing attacks.
Since PHP now requires libxml >= 2.9.0 external entity loading no longer
needs to be disabled to prevent these attacks. It is disabled by default.
Also, the method has an unwanted side effect that causes a lot of
confusion: Parsing XML data from resources like files is no longer possible.

Closes GH-5867.

Bump libxml version requirement 2.7.6 => 2.9.0

Since libxml version 2.9.0 external entity loading is disabled by default.
Bumping the version requirement means that XML processing in PHP is no
longer vulnerable to XXE processing attacks by default.

FPM: Add pm.status_listen option

This option allows getting status from different endpoint (e.g. port
or UDS file) which is useful for getting status when all children are
busy with serving long running requests.

Internally a new shared pool with ondemand process manager is used. It
means that the status requests have reserved resources and should not
be blocked by other requests.

Promote warnings to exceptions in ext/gettext, ext/sysvmsg and ext/xml

Closes GH-5926

[RFC] Only unserialize Phar metadata when getMetadata() is called

In other words, don't automatically unserialize when the magic
phar:// stream wrappers are used.
RFC: https://wiki.php.net/rfc/phar_stop_autoloading_metadata

Also, change the signature from `getMetadata()`
to `getMetadata(array $unserialize_options = [])`.
Start throwing earlier if setMetadata() is called and serialization threw.

See https://externals.io/message/110856 and
https://bugs.php.net/bug.php?id=76774

This was refactored to add a phar_metadata_tracker for the following reasons:
- The way to properly copy a zval was previously implicit and undocumented
  (e.g. is it a pointer to a raw string or an actual value)
- Avoid unnecessary serialization and unserialization in the most common case
- If a metadata value is serialized once while saving a new/modified phar file,
  this allows reusing the same serialized string.
- Have as few ways to copy/clone/lazily parse metadata (etc.) as possible,
  so that code changes can be limited to only a few places in the future.
- Performance is hopefully not a concern - copying a string should be faster
  than unserializing a value, and metadata should be rare in most cases.

Remove unnecessary skip in a test(Compression's unused)

Add additional assertions about usage of persistent phars

Improve robustness of `Phar*->setMetadata()`

- Add sanity checks for edge cases freeing metadata, when destructors
  or serializers modify the phar recursively.
- Typical use cases of php have phar.readonly=1 and would not be affected.

Closes GH-5855

Make http stream wrapper advertise HTTP/1.1 by default

In practice, we always act as an HTTP/1.1 client, for compatibility
with servers which ignore protocol version. Sending the version in
the request will avoid problems with servers which don't ignore it.

HTTP/1.0 can still be forced using a stream context option.

Closes GH-5899.

Merge branch 'levim/assert-throw'

This changes assert.exception to default to 1, or on by default.
Closes GH-5925.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Removed space from bug55086.phpt

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Removed space from bug55086.phpt

Removed space from bug55086.phpt

Closes GH-5923.

Fix #79923: fix socket_set_option_bindtodevice test

Closes GH-5924.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fix bug #79924

Fix bug #79924

Move functions that are only used with ipv6 support under
HAVE_IPV6 checks.

Merge branch 'PHP-7.4'

* PHP-7.4:
  set -ex on travis/compile.sh

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  set -ex on travis/compile.sh

set -ex on travis/compile.sh

-e to fail early: ./configure may fail and further errors from make bury
the original problem in the logs.

-x for easier debugging

Promote "undefined array key" notice to warning

This implements the last remaining part of the
https://wiki.php.net/rfc/engine_warnings RFC.

Closes GH-5927.

Fix fix for bug #79877 for PHP 8

PHP 8 is supposed to throw here.

Don't suppress notices in rfc1867 tests

The references notices in debug builds don't seem to be present
anymore.

Merge branch 'PHP-7.4' into master

* PHP-7.4:
  [ci skip] Add missing NEWS entries

Merge branch 'PHP-7.3' into PHP-7.4

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

[ci skip] Add missing NEWS entries

Convert SPL illegal offset type into TypeError

Make this consistent with the corresponding engine behavior.
Also adjust the messages to match.

Merge branch 'PHP-7.4' into master

* PHP-7.4:
  Fix #79877: getimagesize function silently truncates after a null byte
  Fix #79797: Use of freed hash key in the phar_parse_zipfile function
  Fix #79877: getimagesize function silently truncates after a null byte
  Fix #79797: Use of freed hash key in the phar_parse_zipfile function

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix #79877: getimagesize function silently truncates after a null byte
  Fix #79797: Use of freed hash key in the phar_parse_zipfile function

datetime: new format "p", same as "P" but returning "Z" for UTC

Fix bug #79925

If the expected return type of a magic method is "object", we also
need to allow class types and "static" to comply with covariance.

Enforce __set_state() parameter type

This fixes one of the issues reported in bug #79925. The parameter
type check for this particular method was missed.

Fix handling of nullsafe method in empty()

Fixes oss-fuzz #24627.

Update NEWS

Fix #79877: getimagesize function silently truncates after a null byte

We have to check for NUL bytes if `getimagesize()` has been called.

Fix #79797: Use of freed hash key in the phar_parse_zipfile function

We must not use heap memory after we freed it.

Fix #79877: getimagesize function silently truncates after a null byte

We have to check for NUL bytes if `getimagesize()` has been called.

Fix #79797: Use of freed hash key in the phar_parse_zipfile function

We must not use heap memory after we freed it.

Merge branch 'PHP-7.4' into master

* PHP-7.4:
  Enable libxml entity loader test case on Windows

Enable libxml entity loader test case on Windows

Merge branch 'PHP-7.2' into PHP-7.3

* PHP-7.2:
  Fix #79877: getimagesize function silently truncates after a null byte
  Fix #79797: Use of freed hash key in the phar_parse_zipfile function

Drop the unneeded pointer casting

The standard says that "A pointer to void may be converted to or from a
pointer to any object type". So the casting is unneeded.

REF:
    * c11: http://port70.net/~nsz/c/c11/n1570.html#6.3.2.3p1
    * c99: http://port70.net/~nsz/c/c99/n1256.html

Closes GH-5916

Fixed bug #60302: DateTime::createFromFormat should new static(), not new self()

Also fixes similar issues for DateTimeImmutable::createFromFormat,
DateTime::createFromImmmutable, DateTime::createFromInterface,
DateTimeImmutable::createFromMutable, and
DateTimeImmutable::createFromInterface.

Fix stub hash of ext/ldap

Add a bunch of missing argument types to stubs

Improve a few parameter names in ext/spl

Use the same names which are used by zend functions.

Convert the $ttl parameter of ldap_exop_refresh() to int

Fix #79877: getimagesize function silently truncates after a null byte

We have to check for NUL bytes if `getimagesize()` has been called.

Fix #79797: Use of freed hash key in the phar_parse_zipfile function

We must not use heap memory after we freed it.

Default assert.exception to 1

Bump timeouts, again

Now that we're testing an additional JIT configuration in nightly
builds, the jobs are running close to an hour.

[ci skip]

Ensure correct signatures for magic methods

[skip ci] Fix grammar/typo nits in docs

The PHP error message says "well-formed", not "well formed"

Fixes GH-5920

Add a few missing types to stubs

Bump timeout again

Convert resources to objects in ext/openssl

Closes GH-5860

Co-authored-by: Nikita Popov <nikita.ppv@gmail.com>

Allow overriding completion in `auto_prepend_file`

Currently, it's possible to override `php -a`s completion
functionality to provide an alternative to the C implementation,
with `readline_completion_function()`.

However, that surprisingly gets overridden when called from
`auto_prepend_file`, because those scripts get run before the interactive shell
is started. I believe that not overriding it would be more consistent
with what happens when you override the completion function **after** the
interactive shell.

CLI is the only built-in API that uses this (See discussion in GH-5872).
I believe MINIT and RINIT will only run once when invoked with `php -a`.

Add documentation about the architecture of how php uses readline/libedit

Closes GH-5872

Get rid of empty function entries

Closes GH-5917

Implement named parameters

From an engine perspective, named parameters mainly add three
concepts:

 * The SEND_* opcodes now accept a CONST op2, which is the
   argument name. For now, it is looked up by linear scan and
   runtime cached.
 * This may leave UNDEF arguments on the stack. To avoid having
   to deal with them in other places, a CHECK_UNDEF_ARGS opcode
   is used to either replace them with defaults, or error.
 * For variadic functions, EX(extra_named_params) are collected
   and need to be freed based on ZEND_CALL_HAS_EXTRA_NAMED_PARAMS.

RFC: https://wiki.php.net/rfc/named_params

Closes GH-5357.

Throw ValueError on empty tag in enchant_broker_request_dict()

Change leftover RETURN_FALSE to RETURN_THROWS() from 4095c0a

Promote empty filename to ValueError in BZ2 extension

ValueError for empty path in stream code

Closes GH-5902

Warning to ValueError promotion in Intl extension Part 1

Affects:
- IntlCalendar
- IntlGregorianCalendar
- IntlBreakIterator

Closes GH-5669

Update i386 job to Ubuntu 20.04

Also run function jit tests on macos

Make check in RECV_VARIADIC more precise

Fetch arg_info only once (it's always the same one...) and check
ZEND_TYPE_IS_SET on it, rather than checking if *any* parameter
has a type.

Fixed JIT when result of ASSIGN is in register, but the original variable might need to be destroyed

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fix #79919: Stack use-after-scope in define()

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix #79919: Stack use-after-scope in define()

Fix #79919: Stack use-after-scope in define()

Instead of the temporary `rv`, we use the `val_free` which is there for
this purpose.

Document the signature change of zend_get_closure_method_def()

Fix the default value handling of imagegif()

Remove more unused func_info fields

We check result type guard in FETCH_DIM_R/IS only if index is long or string.

Add type inference support for RECV_VARIADIC

Remove unused recv_arg_info from func_info

Remove some dead code in RECV type inference

We must always have arg_info corresponding to RECV.

Clear result type for exit point caused by JMPZ_EX/NZ_EX

Fixed memory leaks caused by incorrect AVOID_REFCOUNTING flag

Add missing RETURN_THROWS()

Add a few missing parameter types in stubs

Related to GH-5627

Skip tests leaking because of preloading failure.

Fixed bug #79897: Promoted constructor params with attribs cause crash

This was caused by the attribute AST being used twice, and was fixed by
creating a temporary copy of it (and destroying said copy) when neccesary.

Check non-zero in is_power_of_two()

And assert non-zero in floor_log2().

Fixes DASM_S_RANGE_I in ext/simplexml/tests/021.phpt.