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.

Prepend compiler warning flags to CFLAG instead to append them

Credit to @bwoebi for the idea

Closes GH-6057

Fix FE_FETCH_R_SIMPLE specialization

After a99d08b5d135fffa1d83b08b056e0080d295d863 the type can
include UNDEF. However, UNDEF can only reach FE_FREE, not FE_FETCH.
As such, simply ignore this type.

Avoid null pointer UB in accel blacklist

Pretty annoying...

Promote warnings to exceptions in ext/soap and ext/xmlwriter

Closes GH-5998

Promote warnings to exceptions in ext/tidy

Closes GH-6051

Fix type inference for FE_RESET on invalid type

FE_RESET sets the result to UNDEF in this case. We should infer
some type here, because no type generally implies unreachable
code. In this case SCCP ended up replacing the FE_RESET result
with null, including in FE_FREE.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fix pi node removal when removing predecessor

Add explicit double cast

Avoid precision warning.

Fix pi node removal when removing predecessor

We can't just remove the uses, we need to replace uses.

The test case only fails on master with SSA integrity violations,
but I believe the root issue also existed previously.

Fixed nested init fcall guards.

Code cleanup in mbstring.c

Comment constants in mbfl_consts.h, remove unused ones

These were unused, and almost certainly will never be used:

- MBFL_ENCTYPE_MWC4BE
- MBFL_ENCTYPE_MWC4LE
- MBFL_ENCTYPE_SHFTCODE
- MBFL_ENCTYPE_ENC_STRM

For the latter two, there were some encodings which were marked with these flags;
but nothing ever _checked_ these particular flags.

Add comment on mbstring East Asian Width table

Don't guard mbstring code with #ifdef HAVE_MBSTRING

This is just a very silly feature of mbstring -- you can compile the source files with
HAVE_MBSTRING undefined, and it will all just compile to (almost) nothing. What is the
use of this? Why compile the source files and link against them if you don't want the
mbstring extension? It doesn't make any kind of sense.

Remove redundant includes from mbstring (and make sure correct config.h is used)

Very interesting... it turns out that when Valgrind support was enabled,
`#include "config.h"` from within mbstring was actually including the file "config.h"
from Valgrind, and not the one from mbstring!!

This is because -I/usr/include/valgrind was added to the compiler invocation _before_
-Iext/mbstring/libmbfl.

Make sure we actually include the file which was intended.

Remove useless definition of NULL in mbfl_string.h

If NULL is not defined by the platform, mbfl_defs.h already defines it.

Optimize php_unicode_convert_case (cuts mbstring case conversion time ~15%)

This function uses various subfunctions to convert case of Unicode wchars.
Previously, these subfunctions would store the case-converted characters in
a buffer, and the parent function would then pass them (byte by byte) to
the next filter in the filter chain.

Rather than passing around that buffer, it's better for the subfunctions to
directly pass the case-converted bytes to the next filter in the filter chain.
This speeds things up nicely.

Fix typos in comments in mb_send_mail

Remove unused functions from mbstring

- mbfl_buffer_converter_reset
- mbfl_buffer_converter_strncat
- mbfl_buffer_converter_getbuffer
- mbfl_oddlen
- mbfl_filter_output_pipe_flush
- mbfl_memory_device_output2
- mbfl_memory_device_output4
- mbfl_is_support_encoding
- mbfl_buffer_converter_feed2
- _php_mb_regex_globals_dtor
- mime_header_encoder_feed
- mime_header_decoder_feed
- mbfl_convert_filter_feed

Separate implementation of mb_{en,de}code_numericentity

Rather than using a magic boolean parameter to choose different behavior of
the subfunction, inline it. The code size doesn't really grow anyways. And
soon these will be trimmed down more.

Use new-style argument parsing macros in mbstring.c

Inline unneeded indirection for mbstring memory management

All memory allocation and deallocation for mbstring bounces through a table of
function pointers before going to emalloc/efree/etc. But this is unnecessary.
The allocators are never swapped out. Better to just call them directly.

Additional tests for mbstring extension

Prevent com::__construct() to modify the $server_name array

We switch to `zend_string`s for simplicity, so there's no need to
`convert_to_string()` anymore, what makes the array separation
superfluous.

Closes GH-6040

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fix leak on assign concat of array and empty string

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix leak on assign concat of array and empty string

Fix leak on assign concat of array and empty string

Fix nullsafe operator with delayed oplines

Closes GH-6056.

Add tests for nullsafe operator on delayed oplines

More agressive "packed array" guards insertion

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fix leak on consteval exception in ReflectionClass::__toString()

Fix leak on consteval exception in ReflectionClass::__toString()

Merge branch 'PHP-7.4'

* PHP-7.4:
  Check update constant failure in ReflectionClassConstant::__toString()

Check update constant failure in ReflectionClassConstant::__toString()

Fix build warnings after timelib update

Perform type guard checks before IS_UNDEF checks (check IS_UNDEF during deoptimization)

Fix fetching default value of internal function with userland arginfo

"Fix" in the sense of "not crash". We aren't able to actually
display the default value for this case, as there's no way to
fetch the relevant information right now.

Merge branch 'PHP-7.4' into master

Merge branch 'PHP-7.3' into PHP-7.4

Fixed test case

Update timelib to 2020.02

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

Update timelib to 2020.01

Promote warnings to exceptions in ext/ftp

Closes GH-6054

Merge branch 'PHP-7.4' into master

Merge branch 'PHP-7.3' into PHP-7.4

Update timelib to 2018.04

Fixed bug #80030

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fix bug #80037

Fix bug #80037

If we're accessing an uninitialized typed property and __get is
defined, don't perform a read_property callback, as __get is
supposed to have no effect on uninitialized typed properties.
Usually it doesn't, but by-reference assignments cannot be
performed through read_property.

I'm deleting the test for bug #80039 again, as it doesn't really
make sense anymore with this fix.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Add test for bug #80039

Add test for bug #80039

This has already been fixed by 247105ae1ae2a04608078f7fcfe88dacab9f55a4,
but let's add the additional test case.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fix #80027 Terrible performance using $query->fetch on queries with many bind parameters

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix #80027 Terrible performance using $query->fetch on queries with many bind parameters

Fix #80027 Terrible performance using $query->fetch on queries with many bind parameters

Added new flags that allow skipping param_evt(s) that are not used by drivers,
in a backwards and forward compatible manner. Updated the pgsql, mysql, sqlite
and oci drivers to properly use the new flags. I've left out pdo_dblib, which
doesn't have a param_hook, and pdo_firebird, which seems to be using
PARAM_EVT_NORMALIZE in a wrong context (param type vs event type).

Fix #80027 Terrible performance using $query->fetch on queries with many bind parameters

Added new flags that allow skipping param_evt(s) that are not used by drivers,
in a backwards and forward compatible manner. Updated the pgsql, mysql, sqlite
and oci drivers to properly use the new flags. I've left out pdo_dblib, which
doesn't have a param_hook, and pdo_firebird, which seems to be using
PARAM_EVT_NORMALIZE in a wrong context (param type vs event type).

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fix handling of exception if valid() during yield from

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix handling of exception if valid() during yield from

Fix handling of exception if valid() during yield from

Fixes oss-fuzz #25296.

Correctly report failure in zend_handle_undef_args()

And do the check before increfing the closure object, otherwise
we'd have to release it as well.

Fixes oss-fuzz #25313.

Fix trampoline leak in array_map

Fix bug #80018: Tolerate var_dump() printing floats in exponential notation in tests

Merge branch 'PHP-7.4'

* PHP-7.4:
  Make MAX_IFD_NESTING_LEVEL an actual nesting level

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Make MAX_IFD_NESTING_LEVEL an actual nesting level

Make MAX_IFD_NESTING_LEVEL an actual nesting level

Currently we only ever increment ifd_nesting_level, so this ends up
being a limit on the total number of IFD tags and we regularly get
bug reports of it being exceeded. I think the intention behind this
limit was to prevent recursion stack overflow, and for that we only
need to check actual recursive usage. I've implemented that here,
and dropped the nesting limit down to a smaller value
(which still passes our tests).

However, it seems that we do also need to have a total limit on
the number of tags, as we don't catch some instances of infinite
looping otherwise. Add this as a separate limit with a higher
value, that should hopefully be sufficient.

This is expected to fix a number of bugs:

https://bugs.php.net/bug.php?id=78083
https://bugs.php.net/bug.php?id=78701
https://bugs.php.net/bug.php?id=79907
https://bugs.php.net/bug.php?id=80016

Merge branch 'PHP-7.4'

* PHP-7.4:
  Improve error_handing replacement functions

# Conflicts:
# Zend/zend_API.c

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Improve error_handing replacement functions

Improve error_handing replacement functions

We explicitly skip calls to user_error_handler in EH_THROW mode

Closes GH-6050.

libmagic: Update patch [ci skip]

Signed-off-by: Anatol Belski <ab@php.net>

EXTENSIONS: Update fileinfo maintainership info [ci skip]

Signed-off-by: Anatol Belski <ab@php.net>

NEWS: Add libmagic note [ci skip]

Signed-off-by: Anatol Belski <ab@php.net>

libmagic: Fix VS compat

libmagic: Remove unneeded includes

Signed-off-by: Anatol Belski <ab@php.net>

fileinfo: Followup fixes

Signed-off-by: Anatol Belski <ab@php.net>

fileinfo: Upgrade to libmagic 5.39

Signed-off-by: Anatol Belski <ab@php.net>

Support cli_server.color on Windows

On Windows, we have to check whether stdout is attached to a console,
and whether that console supports VT100 control codes.

Closes GH-5996

Fix call to type error functions from jit

Missed to adjust those calls.

Avoid ubsan warning with dummy cache slot addr

Fix unused variable warning

Remove unnecessary cache_slot arguments

zend_verify_type_error_common() no longer needs the cache_slot,
so drop it there and from all users.

Fix execute fuzzer on i386

Opcode handlers use the FASTCALL calling convention...

Rehash function table after disabling functions

To perform fast shutdown without full table cleanup we need all
internal functions to be in one continuous chunk. This was
violated when functions were deleted via disable_functions.

This drops the zend_disable_function() API in favor of
zend_disable_functions(), which disables the given list of
functions and performs the necessary rehash afterwards.

Also drop PG(disabled_functions), which is no longer used.

Don't mark variadic functions as ZEND_FASTCALL

Variadic functions do not support the fastcall calling convention.

Improve type declarations for Zend APIs

Voidification of Zend API which always succeeded
Use bool argument types instead of int for boolean arguments
Use bool return type for functions which return true/false (1/0)
Use zend_result return type for functions which return SUCCESS/FAILURE as they don't follow normal boolean semantics

Closes GH-6002

Don't load IP when entering inlined function

Fix tracked_realloc

We should only drop the information about the old allocation after
checking the memory limit. This makes the code a bit more awkward...

Promote incomplete class to modification to Error

In line with engine diagnostic promotions, property writes throw
an Error exception, while reads throw a warning.

Promote ArrayObject modification during sorting to Error exception

Throw Error instead of E_ERROR when calling method on incomplete class

There's no reason for this to abort execution completely.

Remove unused BG(str_ebuf) global

Also change BG(strtok_len) to size_t. This stores a string length,
so it should be size_t rather than zend_ulong.

Export and reuse zend_is_valid_class_name API

Unserialization does the same check as zend_lookup_class, so let's
share the same optimized implementation.

Use zend_string when storing incomplete class name

No need to copy the string here...

Store incomplete_class entry as normal global

I don't see any reason why this needs to live in a thread-safe
global, unlikely all other class entries.

Avoid useless IP load

Remove useless register allocations