Merge branch 'PHP-7.4'

Don't overwrite iterator count during real init

Merge branch 'PHP-7.3' into PHP-7.4

Fix HT flags copying wrt iterator count

HT_FLAGS() includes the full flag word, including the iterator
count. When we're fully reassigning it, we need to make sure that
we either really do want to copy the iterator count (as in some
cases in array.c) or we need to mask only the actual flag byte.

Add an assert to hash_iterators_del() to make sure the iterator
count is non-zero (which is how I ran into this) and make sure that
the iterator count is correctly preserved during array splicing.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Prepare main branch for PHP 7.3.6

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Prepare main branch for PHP 7.3.6

Prepare main branch for PHP 7.3.6

Merge branch 'PHP-7.4'

Merge branch 'PHP-7.3' into PHP-7.4

s/mysql_connect()/mysqli_connect() in php.ini

Fix second part of bug #77903

When a HT iterator is one past the end and we rehash, we need to make
sure that it is move to the new one past the end position, to make
sure that newly inserted elements are picked up.

Partial fix for bug #77903

In the hash position APIs, make sure we always advance to the next
non-undef element and not just when the position is 0 (similar to
what foreach does). This can happen when the position of an
ArrayIterator is one past its current end and a new element is
inserted not directly at that position because the array is packed.

There is still a bug here (as shown in the tests), but this is a
separate issue that also affects plain array iteration in foreach.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Use curl_mime_*() functions if available

Use curl_mime_*() functions if available

As of curl 7.56.0, `curl_formadd()` is deprecated in favor of
`curl_mime_*()`, so we use the latter if available.

Merge branch 'PHP-7.4'

* PHP-7.4:

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

* 'PHP-7.4' of git.php.net:php-src:
  Remove not needed extension generated files gitignores
  Add test for curl_version()
  Fixed bug #77895
  Zero sockaddr struct
  Fix saproxy_property_write signature
  Correctly destroy reference in ArrayObject sort

Merge branch 'PHP-7.4'

* PHP-7.4:
  execute_data->return_value is not used for internal functions

execute_data->return_value is not used for internal functions

Merge branch 'PHP-7.4'

* PHP-7.4:
  Remove not needed extension generated files gitignores

Remove not needed extension generated files gitignores

These were refactored and moved to build directory via:
2957651c5c2a003e96318fc9217764b91266723a and
7e445ef3b1a197dccbc92f35cb66988b66671841

Merge branch 'PHP-7.4'

Add test for curl_version()

Both readlink args should not point to the same buffer

Merge branch 'PHP-7.4'

Merge branch 'PHP-7.3' into PHP-7.4

Merge branch 'PHP-7.2' into PHP-7.3

Fixed bug #77895

Merge branch 'PHP-7.4'

Merge branch 'PHP-7.3' into PHP-7.4

Merge branch 'PHP-7.2' into PHP-7.3

Zero sockaddr struct

in6 has a bunch of extra fields that we're leaving uninitialized. I
don't see them locally, but there's valgrind warnings on gcov.

Merge branch 'PHP-7.4'

Fix saproxy_property_write signature

Merge branch 'PHP-7.4'

Correctly destroy reference in ArrayObject sort

The reference may be captured in an exception backtrace, in which
case the refcount may be more than one.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Removed tests that always (or almost always) true

Removed tests that always (or almost always) true

Suppress some maybe uninit warnings in jit

Merge branch 'PHP-7.4'

Try to fix stat_variation3-win32.phpt

After fixing comparison operator typos in
f73c104927b64d6137cbe42bf42260d9349cbd02 this test is failing. I
believe the comparison here should be using < rather than ==.

Fix last maybe uninit warnings on 7.4

Most of these only occur under GCC 5. Not fond of all the workarounds
(especially the PDO one), but it gets us a clean build...

Avoid maybe uninit warning in libmagic

GCC 5 false positive, gone in newer versions.

Merge branch 'PHP-7.4'

Merge branch 'PHP-7.3' into PHP-7.4

Merge branch 'PHP-7.2' into PHP-7.3

Fix uninitialized cert_captured

This is a legimitimate bug and also shows up under valgrind.

Disable strict aliasing warnings

It seems like GCC 5 likes to warn about exactly those trivial strict
aliasing violations that it's not going to miscompile anyway and the
warnings are gone in newer versions. Just disable the noise.

Merge branch 'PHP-7.4'

Merge branch 'PHP-7.3' into PHP-7.4

Merge branch 'PHP-7.2' into PHP-7.3

Fixed bug #77882

Merge branch 'PHP-7.4'

* PHP-7.4:
  Updated NEWS
  Fix a NULL dereferencing crash in php-ldap

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Updated NEWS
  Fix a NULL dereferencing crash in php-ldap

Updated NEWS

Fix a NULL dereferencing crash in php-ldap

This happens when parsing a control response to a VLV request which does
 not contain a "context" string.

See bug #77869

Merge branch 'PHP-7.4'

Remove functions from func_info that were removed in PHP 7

Merge branch 'PHP-7.4'

Fix incorrect compare_stats operator

Merge branch 'PHP-7.4'

* PHP-7.4:
  Remove HAVE_* for always available extensions

Remove HAVE_* for always available extensions

Merge branch 'PHP-7.4'

* PHP-7.4:
  Disable PACKAGE_* preprocessor symbols

Disable PACKAGE_* preprocessor symbols

Autoconf defines PACKAGE_* symbols:
- PACKAGE_NAME
- PACKAGE_VERSION
- PACKAGE_TARNAME
- PACKAGE_STRING
- PACKAGE_BUGREPORT
- PACKAGE_URL

and appends them to the generated config.h.in files. With AC_INIT change
via afd52f9d9986d92dd0c63832a07ab1a16bf11d53 where package version, URL,
bug report location and similar meta data are defined, these
preprocessor macros are then non empty strings in the generated
configuration header file. When using phpize, PHP shares the config
files in extensions, warnings of redefined macros appear, such as:
- `warning: 'PACKAGE_NAME' macro redefined`

This patch now disables these non utilized symbols in the generated
config header files.

Better practice would be to include only API specific headers where
needed but this would require even more refactorings. Some extensions
such as pcre, pgsql, and pdo_pgsql solve this issue by undefining some
of these symbols before including the library configuration headers in
the code also. Because these symbols can be defined by any library which
uses Autotools.

Additionally, the unused PACKAGE_* symbols were cleaned for the bundled
libmbfl library and with this patch not needed undef code removed.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Remove PHP_READDIR_R_TYPE

Remove PHP_READDIR_R_TYPE

The PHP_READDIR_R_TYPE m4 macro has been removed via
2b28f7189144a21e753dbc09efadd571121a82b9.

HAVE_POSIX_READDIR_R in windows header file is also not needed anymore.

Merge branch 'PHP-7.4'

Merge branch 'PHP-7.3' into PHP-7.4

Merge branch 'PHP-7.2' into PHP-7.3

LiteSpeed SAPI 7.3, better process management, new API function litespeed_finish_request().

Merge branch 'PHP-7.4'

Try to fix windows build

noreturn attribute has to be on the declaration as well.

Fix strict aliasing violation in phpdbg

By explicitly computing the message length from bytes. This also
makes sure that the length is interpreted in an endianness-independent
manner.

Don't use php_strtok_r in build_tablename

This is unnecessary convoluted... we just want to find the position
of the dot. Using php_strtok_r requires a copy of the string, and the
implementation also relied on the specific semantics of the scratch
space that strtok_r uses.

Fix uninit warning in gd.c

This matches the upstream implementation.

Avoid uninit warning in ftp

Merge branch 'PHP-7.4'

Remove dubious memcmp optimization in iconv

This violates strict aliasing, and appears entirely unnecessary --
the compiler can optimize fixed size memcmp's by itself.

Avoid uninit warnin in http_fopen_wrapper

This one looks semi-legit, in case php_stream_eof() returns false
but php_stream_get_line() fails. Not totally sure this cannot
happen, so rewriting to check both conditions at once.

Avoid uninit warning in dtoa

This code is extremely convoluted, I have no idea whether or not
this one is legitimate. Suppressing with explicit initialization.

Avoid misc uninitialized variable warnings

Avoid uninitialized variable warnings in gmp

Add test for get_cfg_var with array variable

And fix incorrect variable shadowing in add_config_entry(). However,
the test doesn't hit this case, as it requires a nested array. I'm
not sure if it's possible to produce nested arrays from ini?

Avoid unitialized variable warning in soap

Extract envelope fetch into separate function, so the control
flow becomes more obvious.

Remove dubious memcmp optimization in iconv

This violates strict aliasing, and appears entirely unnecessary --
the compiler can optimize fixed size memcmp's by itself.

Avoid uninit warnin in http_fopen_wrapper

This one looks semi-legit, in case php_stream_eof() returns false
but php_stream_get_line() fails. Not totally sure this cannot
happen, so rewriting to check both conditions at once.

Avoid uninit warning in dtoa

This code is extremely convoluted, I have no idea whether or not
this one is legitimate. Suppressing with explicit initialization.

Avoid misc uninitialized variable warnings

Avoid uninitialized variable warnings in gmp

Add test for get_cfg_var with array variable

And fix incorrect variable shadowing in add_config_entry(). However,
the test doesn't hit this case, as it requires a nested array. I'm
not sure if it's possible to produce nested arrays from ini?

Avoid unitialized variable warning in soap

Extract envelope fetch into separate function, so the control
flow becomes more obvious.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Include libpq-fs.h on Windows, too

Include libpq-fs.h on Windows, too

That's more future proof than defining hard-coded macros.

Merge branch 'PHP-7.4'

* PHP-7.4:
  Fix tests wrt. internationalization

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix tests wrt. internationalization

Merge branch 'PHP-7.2' into PHP-7.3

* PHP-7.2:
  Fix tests wrt. internationalization

Fix tests wrt. internationalization

Merge branch 'PHP-7.4'

Fix SOAP bailout handling

This code was reusing the _bailout variable from
SOAP_CLIENT_BEGIN/END_CODE(). As _bailout is not volatile, modifying
it after the setjmp call and then reading it back on return is
illegal. Use a separate local bailout variable instead.

This fixes the miscompile introduced by marking zend_bailout() as
noreturn.

Enable -Wall on release builds for GCC

-Wall was already enabled for debug builds, enable it for release
builds as well.

Merge branch 'PHP-7.4'