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

Merge branch 'PHP-7.4'

* PHP-7.4:
  Unset ai_canonname field in dump

Unset ai_canonname field in dump

Okay, looks like ai_canonname does not get set on macos even if
AI_CANONNAME is specified, so canonicalize this in the other
direction instead and unset ai_canonname if it is present.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Check ps -p availability in proc_nice test
  Set AI_CANONNAME flag in socket_addrinfo test
  Add ipv6 skipif to test
  Improve privilege check in pcntl_setpriority() test

Check ps -p availability in proc_nice test

Set AI_CANONNAME flag in socket_addrinfo test

Musl always populates ai_canonname. Set the flag to avoid output
discrepancies.

Add ipv6 skipif to test

This was done for the -unix variant, but not the (formerly
windows-only?) main test.

Improve privilege check in pcntl_setpriority() test

We need CAP_SYS_NICE privileges, which might not be available
just because we're running as root (Docker...)

Merge branch 'PHP-7.4'

* PHP-7.4:
  Suppress warning in pcntl_unshare skipif
  Use standard types in fopencookie test
  Remove posix_kill() test
  Relax error message check
  Check whether setlocale -a is available

Suppress warning in pcntl_unshare skipif

Use standard types in fopencookie test

Instead of internal __size_t / __off64_t types use ssize_t and off64_t.
This makes it work on musl as well.

Remove posix_kill() test

This test is doing something really dubious. There's a race
condition here that would kill some poor process that happens
to start at just the wrong time.

Relax error message check

Check whether setlocale -a is available

Suppress warning in pcntl_unshare skipif

Use standard types in fopencookie test

Instead of internal __size_t / __off64_t types use ssize_t and off64_t.
This makes it work on musl as well.

Remove posix_kill() test

This test is doing something really dubious. There's a race
condition here that would kill some poor process that happens
to start at just the wrong time.

Relax error message check

Check whether setlocale -a is available

Update NEWS for 8.0.0beta2

Update API version numbers

In preparation for PHP 8.0.

Revert "Prepare for PHP 8.1"

This reverts commit 1ab4d0e6b7abb67d5a01ce3327bf973772271fb2.
This reverts commit a359635cb1a4df8b5137a506c88c4cb102acac0e.

FPM: Prevent warning with raising pm.max_children for shared pool

Prepare for PHP 8.1

Update API version numbers

In preparation for PHP 8.0.

Relax precision in atan test

Relax precision in hypot test

Add compatibility define for PHP_CHECK_GCC_ARG

This defines the m4 macro, but makes it always error.

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.