Improve error messages mentioning parameters instead of arguments

Closes GH-5999

Fixed memory leak

Wildcard output differences in snmp tests

And point out that snmp-mibs-downloader is needed.

Tracin JIT support for FETCH_DIM_W/RW with IS_VAR + IS_INDIRECT first operand.

Avoid use of remote_port in snmp

This field is not used (and has not been used for a long time --
I've seen some mailing list thread from 2003 about it!) and throws
a deprecation warning.  The port is part of peername instead (for
transports that support a port at all).

Pass correct op_info to zend_jit_fetch_dim()

Flesh out ZEND_HASH_REVERSE_FOREACH_* macros

Closes GH-6080.

Fix 3b0fecd and resulting test fail lob_012.phpt

JIT for FETCH_DIM_W/RW insructions

Fix Windows build for shared ext/sockets

We can't use `PHPAPI` for extensions which may be built shared.  Thus,
we introduce `PHP_SOCKETS_API`.

Don't handle missing value in zend_verify_arg_error()

This error condition is handled separately, never reaches this
function.

mb_str_split is already documented on php.net

So remove TODO comment which implies that it's not.

JIT for IN_ARRAY instruction.

Merge branch 'PHP-7.4' into master

* PHP-7.4:
  Fix #72941: Modifying bucket->data by-ref has no effect any longer

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix #72941: Modifying bucket->data by-ref has no effect any longer

Fix #72941: Modifying bucket->data by-ref has no effect any longer

To match the PHP 5 behavior, we have to explicitly cater to `buffer` or
`data` being references.

Closes GH-6096.

Try to fix 32-bit setcookie tests, again

add socket_ce and socket_import_file_descriptor in public API

Fix some snmp stubs

As well as some basic mistakes in tests.

Suppress unused variable in snmp

force_ipv6 may be unused if compiling without ipv6 support.

Run the opcache-only configuration only for scheduled builds

The opcache-only configuration has very little signal (i.e. it is
very rare that it fails while non-opcache and opcache+jit both
pass). Switch it to run only for nightly builds, so we get faster
results on normal builds.

Try to fix setcookie() tests on 32-bit

micro-optimization

JIT for FETCH_LIST_R

Fix UNKNOWN default values in ext/oci8

Closes GH-6089

Make null byte error a ValueError

Currently we treat paths with null bytes as a TypeError, which is
incorrect, and rather inconsistent, as we treat empty paths as
ValueError. We do this because the error is generated by zpp and
it's easier to always throw TypeError there.

This changes the zpp implementation to throw a TypeError only if
the type is actually wrong and throw ValueError for null bytes.
The error message is also split accordingly, to be more precise.

Closes GH-6094.

Always use PCRE for mbstring.http_output_conv_mimetypes

Instead of using either oniguruma or pcre depending on which is
available. We always have PCRE, so use it. This ensures consistent
behavior.

Promote warnings to errors for set(raw)cookie()

Closes GH-5819

Merge branch 'PHP-7.4'

* PHP-7.4:
  Support more placeholders in bless_tests.php

Support more placeholders in bless_tests.php

And don't replace trailing --CLEAN-- sections.

Require $method parameter in openssl_seal/openssl_open

RC4 is considered insecure, and it's not possible to change the
default of these functions. As such, require the method to be
passed explicitly.

Closes GH-6093.

Promote warnings in exif

The only thing that can promoted are the path-related checked.
Everything else is input dependent and error-suppressing these
functions is both the typical and the recommended usage.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Revert "Update libzip on macos"
  Revert "Manually build re2c on macos"

Revert "Update libzip on macos"

This reverts commit 090bddb93488b6bcf411e970afe77d96f86f4ac8.

Revert "Manually build re2c on macos"

This reverts commit 9bbe236f6adead060d83c87a581674f4917db3de.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fixed bug #80077

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fixed bug #80077

Fixed bug #80077

Quoting from the bug report:

> The domain names passed to getmxrr() do not contain a trailing dot.
> DNS lookups which do not find records will (depending on the local
> resolver config) try again by adding the local domain to the end of
> the searched host/domain. In many environments there's an mx record
> for any subdomain of the local domain and the MX query will return
> a hit. But the test expects no hit. So the test fails when checking
> that "qa.php.net" does not have an MX record in DNS. In our local
> environment the resolver falls back to also check qa.php.net.kippdata.de
> which does have an MX record. Using "qa.php.net." instead of "qa.php.net"
> should fix this for everyone.

decbin/decoct/dechex optimization.

Catch type errors so test completes and doesn't diff

Accept updated error number generated by the latest Oracle version

Yes, and mask the dir path too.

Fix parsing regression from PHP 7

Adjust assignment line number for match

Otherwise the assignment will have the same number as the default arm
which will 1. mis-trigger a breakpoint and 2. mark the line as covered
even when it isn't.

Closes GH-6083

Reduce input size limit in execute fuzzer

We only have 4 Zend test cases > 8k. Large inputs tend to just
make things slower.

Add "const". Move constant strings to read-only memory.

Promote warnings to exceptions in ext/intl

Closes GH-5972

Don't allow dynamic properties on generators

Noticed this because we leak those properties in GC. This was
never intended to be allowed.

Fix UNKNOWN default values in various extensions

Closes GH-6075

Fix UNKNOWN default values in ext/standard

Closes GH-6026

Merge branch 'PHP-7.4'

* PHP-7.4:
  Don't leave behind temporary file in bug70362.phpt

Don't leave behind temporary file in bug70362.phpt

Drop support for crypt() without explicit salt

crypt() without salt generates a weak $1$ MD5 hash. It has been
throwing a notice since 2013 and we provide a much better alternative
in password_hash() (which can auto-generate salts for strong
password hashes), so keeping this is just a liability.

Allocate temporary PCRE match data using ZMM

Create a separate general context that uses ZMM as allocator and
use it to allocate temporary PCRE match data (there is still one
global match data). There is no requirement that the match data
and the compiled regex / match context use the same general context.

This makes sure that we do not leak persistent memory on bailout
and fixes oss-fuzz #25296, on which half the libfuzzer runs
currently get stuck.

Avoid large eval inputs in fuzzer

While we limit the size of the main compilation input, the size
of eval inputs was not limited. This could result in stack
overflows, e.g. oss-fuzz #25464.

Accept zend_string instead of zval in zend_compile_string

Private/public split curl header

To allow exporting the php_curl.h header containing curl class
entries, split off a separate curl_private.h header with all the
implementation details.

We may move or expose additional APIs in php_curl.h on an as-needed
basis.

Remove some unnecessary HAVE_EXTNAME guards

A recurring pattern in old extension: Putting the whole source
code behind HAVE_EXTNAME. This is pointless, as the code is only
compiled if the extension is enabled.

This removes a couple of them, but not all.

Opcache JIT, code simplification for Haiku.

More straightforward approach to get the path of the
 current PHP process.

Closes GH-6082.

Add phpunit to community project tests

Throw on invalid mb_http_input() type

Fix incorrect/unused macro

zend_ts_hash_init accepts 4 arguments, not 5.
The pHashFunction parameter was removed in 5d2576264653c2faaca9cd7d64218d10ab612408

Closes GH-6087

Improve handling of `#[` attributes in `php -a`

`php -a` treats lines starting with `#` as comments when deciding if
the provided statement is valid.

So it passed `#[MyAttr]` to the parser after the user hits enter,
causing a syntax error for multi-line statements..

With this patch, the following snippet is parsed correctly

```
php > #[Attr]
php > function x() { }
php > var_export((new ReflectionFunction('x'))->getAttributes()[0]->getName());
'Attr'
```

Followup to GH-6085

Closes GH-6086

Improve handling of `#[` in `php -a`

PHP treats `#ini_setting=value` as a call to
`ini_set('ini_setting', 'value')`,
and silently skips undeclared settings.

This is a problem due to `#[` becoming supported attribute syntax:

- `#[Attr] const X = 123;` (this is not a valid place to put an attribute)
  This does not create a constant.
- `#[Attr] function test($x=false){}` also contains `=`.
  This does not create a function.

Instead, only treat lines starting with `#` as a special case
when the next character isn't `[`

Closes GH-6085

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fix azure i386 build

Fix azure i386 build

Looks like pgsql on i386 broke even more.

Handle null encoding in mb_http_input()

Reenable s390x on travis

Only enable it for cron builds. We don't need to run exotic
architectures on every commit and PR.

Closes GH-6076.

Fixed bug #80057 (DateTimeImmutable::createFromFormat() does not populate time)

Promote warnings to exceptions in ext/ldap

Closes GH-6065

Use ZPP instead of custom type checks

We can add these types as a native type declaration to stubs as a side-effect. Closes GH-6068

Use the canonical order of types in array|string ZPP error messages

Add the Z_PARAM_ARRAY_HT_OR_NULL and Z_PARAM_OBJ macros

Extract common flock code

As SPL is currently a copie of the code in file.c

Closes GH-6069

Release call trampolines in zpp fcc

When using zpp 'f' or Z_PARAM_FUNC, if the fcc points to a call
trampoline release it immediately and force zend_call_function
to refetch it. This may require additional callability checks
if __call is used, but avoids the need to carefully free fcc
values in all internal functions -- in some cases this is not
simple, as a type error might be triggered by a later argument
in the same zpp call.

This fixes oss-fuzz #25390.

Closes GH-6073.

Update travis to bionic

Make gethostbyname() test more liberal

This returns 127.0.1.1 on travis bionic.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Avoid duplicate octal warning during heredoc scan ahead

Disable InfiniteIterator class while fuzzing

The combination of LimitIterator and InfiniteIterator can cause
effectively infinite loops that bypass the executor step limit.

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Avoid duplicate octal warning during heredoc scan ahead

Avoid duplicate octal warning during heredoc scan ahead

Avoid unnecessary IP initiliaization on trace linking

Cleanup macro/function names

Load EX(opline) in one instuction if possible

Reorder DynAsm macros (no other changes)

Error promotions in SPL

Warning to Error promotion and a Notice to Warning promotion to align
with the behaviour specified in the Reclassify Engine Warnings RFC.

Closes GH-6072

Warning to Error promotion in ext/standard

Those should be the last ones other than set(raw)cookie()

Closes GH-5814

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fixed bug #80049

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fixed bug #80049

Fixed bug #80049

Type checking may convert to refcounted values, so force freeing
of extra args.

If we don't know the return address, just escape to VM, instead of adding side exit.
Remove unnecessary exception checks.

Use symbolic constants in Japanese kana conversion code (not magic numbers)

Also correct misspelling of 'hiragana' as 'hirangana' at the same time.

Remove unused 'from' field from mbfl_buffer_converter struct

Add comment to mbfilter_tl_jisx0201_jisx0208.h

Explain the 'ZEN' and 'HAN' in symbolic constant names.

Remove unneeded function mbfl_filt_ident_common_dtor

This was the default destructor for mbfl_identify_filter structs, but there's nothing
we actually need to do to those structs before freeing them.

Remove unneeded function mbfl_filt_conv_common_dtor

This is a default destructor for mbfl_convert_filter structs. The thing is: there
isn't really anything that needs to be done to those structs before freeing them.
The default destructor just zeroed out some fields, but there's no reason why
we should actually do that.

Refactor mbfl_convert.c

Merge branch 'PHP-7.4' into master

* PHP-7.4:
  Fix #80048: Bug #69100 has not been fixed for Windows

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix #80048: Bug #69100 has not been fixed for Windows

Fix #80048: Bug #69100 has not been fixed for Windows

We fix the erroneous length calculation on Windows, too.

Closes GH-6067.