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

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.

Include libpq-fs.h on Windows, too

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

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

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.

Fix unused variable warnings in libmagic

These variables are only used in assertions.

Remove odd variable assignment in escape_analysis

As we are breaking out of the for loop setting i = ssa_vars_count
doesn't do anything, but it confuses GCC into throwing a no-overflow
assumption warning.

Fix potentially uninitialized warnings in phpdbg

Regroup expressions in php_schema.c to avoid array bounds warnings

Avoid potentially-uninitialized warnings in readline

Fix uninitializde heredoc_tag use in readline

Could happen if "<<<" is directly followed by a newline.

Mark zend_bailout() as a noreturn function

Merge branch 'PHP-7.3' into PHP-7.4

Merge branch 'PHP-7.2' into PHP-7.3

Fix key leaks in mb_convert_encoding()

Use EMPTY_SWITCH_DEFAULT_CASE in php_unicode.c

Avoids a potentially uninitialized variable warning.

Run compile step under travis_wait

If there are no warnings, we may end up compiling more than 10
minutes without output, in which case the build times out. Use
travis_wait to avoid this.

Avoid directory clash in realpath tests

Backported call frame initialization improvement

Replace "ZEND_CALL_CTOR" hack by additional live-range

Added test

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix potential OPcache file cache related issues

Merge branch 'PHP-7.2' into PHP-7.3

* PHP-7.2:
  Fix potential OPcache file cache related issues

Fix potential OPcache file cache related issues

To solve issues detected during testing, we backport the following
commits to PHP 7.2:

129c5c1181bf344b37e13fd6dc5dfe76c13d7208
9ac133a0b3863ca4d9659140154ee237e5f4669a
ce72bc6b658c335dd37393d0beb28584e6805e97

Work around -Walloc-size-larger-than bug

Fixed typo

Use readdir() instead of readdir_r()

readdir_r() is deprecated in modern glibc versions. readdir() is
thread safe in practice, as long as there are no concurrent accesses
on the *same* directory stream.

Remove curl gnutls locking

This is only relevant for the gcrypt backend, which is no longer in
use. I couldn't find any hard info on when exactly gcrypt support was
dropped, but nette is the default since 2.11.1. Most code that still
has the gcrypt handling is checking against < 2.12. In either case,
those versions were released in 2010/2011.

Fix libgd warnings

Remove incorrect mbstring includes

And thus fix unused variable warnings.

Remove unused function prototype

Remove restrict zend_portability define

We're not using this anywhere, and it causes warnings, so I'm
dropping it. If we do end up needing this, it should be reintroduced
with a ZEND_ prefix.

Fix error message in spl_autoload test

Fix libmagic warnings

Add script to generate libmagic.patch

Remove the readline thread-safety warning

Would be nice to have this in configure, but we'd have to change
the ordering of extension & Zend processing for that.

Fix pointer signedness warning in mysqlnd

Remove unused rcsid variables

Avoid curl ssl callback warnings with OpenSSL 1.1

Fix typo

Fixed bug #77877 (call_user_func() passes $this to satatic methods).

Merge branch 'PHP-7.3' into PHP-7.4

Avoid onig match param unused variable warning

Suppress warnings using zend_quiet_write

Generate ZEND_COUNT for sizeof()

sizeof() is an alias of count(), so we should generate the same
code for them.

Make tsrm_env_lock() void

We're not checking the return value and the NTS version of this
generates warnings. If we want to handle lock failures, we should
do a hard abort inside tsrm_env_lock() itself.

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Ensure double slashes are replaced by the path normalization

Support VirtualProtect for opcache.protect_memory

Don't enable this on AppVeyor yet, as there is still an open
issue in phar.

Ensure double slashes are replaced by the path normalization

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Fix phar:// include handling with file cache

Merge branch 'PHP-7.2' into PHP-7.3

* PHP-7.2:
  Fix phar:// include handling with file cache

Fix phar:// include handling with file cache

Remove posix_getlogin() check from posix_getpwnam() test

It doesn't make sense to pass false to posix_getpwnam(). I'm not
sure what exactly this is guarding against, but it won't work in
this form.

Make zlib include_path tests more robust

One file is placed inside the script directory, we need to make
sure it has a unique name as well.

Merge branch 'PHP-7.3' into PHP-7.4

Merge branch 'PHP-7.2' into PHP-7.3

Fix 29nb_async_connect.phpt

Intermediate states are unreliable across versions (I think), make
sure we go through the whole connect procedure.

Merge branch 'PHP-7.3' into PHP-7.4

Merge branch 'PHP-7.2' into PHP-7.3

Fix pgsql use after free trying to reuse closed connection

When a connection is closed, we also need to remove the hash entry
from the regular_list, as it now points to freed memory. To do this
store a reverse mapping from the connection to the hash string.

It would be nicer to introduce a wrapping structure for the pgsql
link resource that could store the hash (and notices), but that would
require large changes to the extension, so I'm going for a more
minimal fix here.

fix a spell mistake

possbile => possible

Merge branch 'PHP-7.3' into PHP-7.4

Merge branch 'PHP-7.2' into PHP-7.3

Fix use after free on pg_close() of default connection

Use release for regex in imap

The regex may be used as a cache key now.

Fix bug #77866: Port Serializable SPL classes to use __unserialize()

Payloads created using Serializable are still supported.

Fixed bug #77873

Fix leak on error in new serialization mechanism

Fixed ext/ffi/tests/044.phpt failure on 32-bit Linux

Eliminated check for assignment to itself

Micro-optimization

Remove duplicated substitude for Makefile

The substitution is already done in the CLI's config.m4 file. Current
SAPIs only provide one PHP_EXECUTABLE variable, i.e. PHP CLI so the one
in the configure.ac can be removed.

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 HAVE_STRERROR

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  Repare SWITCH VM

Merge branch 'PHP-7.2' into PHP-7.3

* PHP-7.2:
  Repare SWITCH VM

Repare SWITCH VM

Remove HAVE_STRERROR

Checking for the strerror function presence is no longer needed since it
is part of the C89 standard [1] and can be safely assumed that all
current systems have it.

Check in the configure.ac and Windows defined symbol are still left
until the file library (libmagic) will be updated.

[1]: https://port70.net/~nsz/c/c89/c89-draft.html

Added range options to the float validation filter

php_filter_int (called via the constant FILTER_VALIDATE_INT) has the options min_range
and max_range. they allow the user to not only test if a value is a double but also if
the value is inside a specific range. php_filter_float (called via the constant
FILTER_VALIDATE_FLOAT) didn't provide this yet, making validation of numeric but
not-only-int values more complicated for the user.
this commits implements the options min_range and max_range for the function
php_filter_float to fix this inconsistency.

Merge branch 'PHP-7.3' into PHP-7.4

Merge branch 'PHP-7.2' into PHP-7.3

Preserve keys in emulate_read_fd_set()

Keys are already preserved in the non-emulated case.

Merge branch 'PHP-7.3' into PHP-7.4

Merge branch 'PHP-7.2' into PHP-7.3

Fixed bug #77853

Merge branch 'PHP-7.3' into PHP-7.4

Merge branch 'PHP-7.2' into PHP-7.3

Fixed bug #77844

We should probably return an integer result from the operation in
typed mode, right now the result is always a string.

Fix invalid function by-ref prop assign handling