Fixed bug #79927

We need to unset the AT_FIRST_YIELD flag when yielding from an
array as well.

In the interest of being conservative, I'm applying this only to
PHP 8.

Fix nullsafe operator on $this

SAVE_OPLINE in NULL_HANDLER

Let's avoid crashing before the nice error message gets printed...

Fix nullsafe operator on reference

Dereference the value before checking the type. As the happy path
necessarily has to check for references, I'm not bothering to
delay the comparison.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fixed bug #79917

Fixed bug #79917

op_arrays can be shared on two levels: Either the op_array is
completely shared, or it is distinct but shares all members
(apart from static_variables).

The the op_array is distinct, we need to make sure to properly
initialize the MAP_PTR structures.

Autogenerate function entries for COMPersistHelper

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

Remove practically unused parameter

The `cached` out parameter of `php_com_load_typelib_via_cache()` was
meant to signal whether a particular typelib actually has been cached.
This is not really relevant, though, for the imagined purposes, and
since the parameter is no longer really used, we removed it altohether.

Merge branch 'PHP-7.4' into master

* PHP-7.4:
  Fix #48585: com_load_typelib holds reference, fails on second call

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix #48585: com_load_typelib holds reference, fails on second call

Fix #48585: com_load_typelib holds reference, fails on second call

Whether the type library is cached is actually irrelevant here; what
matters is that the symbols are imported, and since these are not
cached, we have to import them for every request.  And we cannot cache
the symbols, because the import depends on the current codepage, but
the codepage is a `PHP_INI_ALL` setting.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fixed bug #79951

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fixed bug #79951

Fixed bug #79951

One branch did not release tmp_replace_entry_str.

Also reduce the scope of some variables.

Remove unnecessary function entry array

Merge branch 'PHP-7.4'

Merge branch 'PHP-7.3' into PHP-7.4

Make sure string is NUL byte terminated.

Fix signature of callback function

Cf. <https://docs.microsoft.com/en-us/windows/win32/api/imagehlp/nc-imagehlp-pimagehlp_status_routine>.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Check the correct list

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Check the correct list

Check the correct list

This was supposed to check mx_list, not weight_list... oops.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fix bug #79944

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix bug #79944

Fix bug #79944

Only return true from dns_get_mx if we actually found any MX record.

Skip bug70417.phpt if lsof output looks wrong

On alpine lsof accepts -p but it doesn't actually do anything.
Add a crude check for whether lsof looks like Linux lsof.

Check permissions in pcntl_unshare_03.phpt

Don't check specific message in socket_shutdown.phpt

Only test one error in socket_strerror() test

The messages differ by used libc and version, so let's not try to
do exhaustive testing here.

Fixed bug #62294

The primary issue was already resolved in 7c3e487289ec41e560cf7a77e36eb43da2234f33,
but the particular example used in this bug report ran into an
additional issue on PHP 8, because I forgot to drop a number of
zend_bailout calls when switch require failure to throw.

Fixed bug #65275

Make EG(exit_status) the single source of truth for the exit status,
instead of having two variables that we cannot really keep
synchronized.

Merge branch 'PHP-7.4' into master

* PHP-7.4:
  Fix #79922: Crash after multiple calls to xml_parser_free()

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix #79922: Crash after multiple calls to xml_parser_free()

Fix #79922: Crash after multiple calls to xml_parser_free()

We must not call `zend_list_delete()` in resource closer functions
exposed to userland, because decreasing the refcount there leads to
use-after-free scenarios.  In this case, commit 4a42fbb worked for
typical use-cases where `xml_parser_free()` has been called exactly
once for the resource, because there is an internal zval (`->index`)
referencing the same resource which already increased the refcount by
one.  However, when `xml_parser_free()` is called multiple times on the
same XML parser resource, the resource would be freed prematurely.

Instead we forcefully close the resource in `xml_parser_free()`.  We
also could decrease the refcount of the resource there, but that would
require to call `xml_parser_free()` which is somewhat uncommon, and
would be particularly bad wrt. PHP 8 where that function is a NOP, and
as such doesn't have to be called.  So we do no longer increase the
refcount of the resource when copying it to the internal zval, and let
the usualy refcounting semantics take care of the resource destruction.

[1] <http://git.php.net/?p=php-src.git;a=commit;h=4a42fbbbc73aad7427aef5c89974d1833636e082>

Remove some unnnecessary null checks in openssl

Remove null checks before EVP_PKEY_free and BIO_free. NULL is a
no-op for both of these. Probably applies to most other freeing
function as well...

Improve X509_PKEY management in OpenSSL

Remove the free_pkey argument from php_openssl_pkey_from_zval,
instead return an EVP_PKEY that always needs to be freed
(by incrementing refcount if necessary).

This makes the code simpler and fixes a number of bugs in the
existing handling.

Closes GH-5946.

Expand upgrading note for namespaced name change

Mention the case from bug #79942.

[ci skip]

Fixed bug #79948

Make sure we don't execute further scripts if one of them encountered
an exit exception.

Also make sure that we free file handles that end up unused due to
an early abort in php_execute_scripts(), which turned up as an
issue in the added test case.

Finally, make use of EG(exit_status) in the places where we
zend_eval_string_ex, instead of unconditionally assigning exit
code 254. If an error occurs, the error handler will already set
exit status 255.

Fixup phar test

Fixed bug #77561

Unconditionally strip shebang lines when using the CLI SAPI,
independently of whether they occur in the primary or non-primary
script. It's unlikely that someone intentionally wants to print
that shebang line when including a script, and this regularly
causes issues when scripts are used in multiple contexts, e.g.
for direct invocation and as a phar bootstrap.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fixed bug #79947

Fixed bug #79947

Move the FREE_OP for op_data out of the zend_binary_assign_op_dim_slow()
slow path, so it can be used by the other error path as well. This
makes ASSIGN_DIM_OP structurally more similar to ASSIGN_DIM.

Fixed bug #79946

Declare __STDC_CONSTANT_MACROS and __STDC_FORMAT_MACROS via -D
to make sure they are declared before the first stdint.h include.
We also define these in php_stdint.h, but don't always include that
file first.

This is necessary for old compilers that use C99 rather than C11
semantics for stdint.h.

proc_open/openpty: DragonFlyBSD build fix.

Similarly to NetBSD emalloc-ish api had been introduced into libutil.

Closes GH-5960.

Revert "Make diff section contents red(-)/green(+) in run-tests.php"

This reverts commit 58489bd3b93e10c150933eefac05297b48213eb3.

This also uses ANSI escape code coloring for .diff files, where it
is not meaningful.

Make diff section contents red(-)/green(+) in run-tests.php

This uses green for lines with `+` and red for lines with `-`.

Colors(Red and Green) would make the failure causes stand out visually when scrolling through errors.

Closes GH-5965

Use short array values in run-tests.php

The run-tests.php file already requires php 7 syntax.
Done automatically with phpcbf with no other changes done/planned.

This file is already using short arrays in other places.

run-tests has already been substantially refactored in PHP 8.0
compared to PHP 7.4.

This makes merge conflicts more likely, but changes to run-tests.php on older
versions are less frequent bug fixes.

Add many missing closing PHP tags to tests

Closes GH-5958

Fix m4 quotation

We should not try to expand AX_CHECK_COMPILE_FLAG inside the
error message...

Promote various warnings to Error in GMP extension

This gets rid of most false returns

Closes GH-5882

Add colours to test results in run-tests.php

The test runner will attempt to colourise the result of each test

The --no-color option is introduced to disable this feature.

Closes GH-5901

Drop '#if 0' in mysqlnd_debug() from the MySQLnd extension

Closes GH-5587

Remove some duplicate iconv path logic

PHP_SETUP_ICONV already adds the include path to INCLUDES and the
library path to SHARED_LIBADD.

Remove PHP_ICONV_H_PATH

Directly include <iconv.h> instead, now that the giconv.h
distinction no longer exists.

Remove giconv support

The best information I was able to find about this is this mail
from 2009 which indicates that giconv is an old FreeBSD iconv
implementation that has long since been superseded by libiconv.

https://lists.samba.org/archive/samba-technical/2009-June/065177.html

Remove more leftovers of PHP_DEFINE handling

opcache gdb: Set the EI_OSABI value for NetBSD.

Closes GH-5948.

Add another round of missing parameter types to stubs

Closes GH-5950

Fix ZPP order in ext/phar

Add UPGRADING.INTERNALS notes for zend_object* changes

Closes GH-5953.

Accept zend_object* in zend_update_property

Accept zend_object* in zend_unset_property

Accept zend_object* in zend_get_exception_base

Accept zend_object in zend_read_property

Merge branch 'PHP-7.4' into master

* PHP-7.4:
  Prevent tempnam tests conflicts

Prevent tempnam tests conflicts

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fix bug #70574: Move files to proper locale dir

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix bug #70574: Move files to proper locale dir

Fix bug #70574: Move files to proper locale dir

Closes GH-5940.

Don't use PHP_DEFINE in iconv

iconv is the only consumer of this custom define mechanism that
puts every define into a single file. Use the standard mechanism
instead.

Add more argument types to stubs

Closes GH-5943

Reapply "Explicitly validate popen mode"

To avoid behavior differences due to libc. This time with the
check only for the non-win32 case, as Windows support additional
modifiers here (t/b).

Revert "Explicitly validate popen mode"

This reverts commit ab36540bddb63139c3da9a9da5e5feacdf7bf7bf.

Causes a test failure on Windows.

Add type information to test-runner

Closes GH-5942

Promote warnings to Error in FileInfo extension

Closes GH-5914

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fix file clash in zlib test

Fix file clash in zlib test

Mention security in step 3, as otherwise it might be overlooked

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fix #79934: CRLF-only line in heredoc causes parsing error

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix #79934: CRLF-only line in heredoc causes parsing error

Fix #79934: CRLF-only line in heredoc causes parsing error

Fixes the function `next_newline()` in zend_language_scanner.l. The
function now correctly returns a newline_len of 2 for "\r\n".

Closes GH-5944.

Avoid getThis() usages in zip extension

There isn't any magic going on here, we can just use ZEND_THIS
and assume it exists.

Explicitly validate popen mode

To avoid behavior differences due to libc.

Add GLOB_BRACE skipif

Temporarily disable s390x job

These aren't getting scheduled and just keep hogging jobs so
nothing else gets scheduled either.

Prefer strtoll over atoll

Both are specified by C99, but strtoll has specified overflow
behavior while atoll does not, so prefer using it.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Check ps -p availability in process title test
  Add privilege check in pcntl_unshare test

Check ps -p availability in process title test

Add privilege check in pcntl_unshare test

Privileges for CLONE_NEWPID were not checked.

Only force log startup errors if display_startup_errors disabled

Otherwise this results in duplicate errors.

Closes GH-5941.

Normalize error message capitalization

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fix memory leak on socket_sendmsg() / socket_recvmsg() error

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix memory leak on socket_sendmsg() / socket_recvmsg() error

Fix memory leak on socket_sendmsg() / socket_recvmsg() error

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fixed bug #79930
  Fix iov_base pointer type for illumos
  Backport bless_tests.php changes from PHP 8

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fixed bug #79930

Fixed bug #79930

We're inserting src_zval, so that's what we should addref.

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix iov_base pointer type for illumos

Fix iov_base pointer type for illumos

These systems has two versions of the iovec interface dependent on
compiler flags passed, the legacy version causing little build issue.

Closes GH-5939.

Backport bless_tests.php changes from PHP 8