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.

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.