serialize() optimization

Optimize out zend_strpprintf() usage for simple concationaton

Improve SPL directory and stat() cache using zend_srting* instead of char*

Merge branch 'PHP-8.0'

Merge branch 'PHP-7.4' into PHP-8.0

Fix memleak on ReflectionFunction and ReflectionGenerator classes when calling __construct after instantiation

Remove quicktester

This is barely used and more of a hinderence than anything else

Closes GH-6712

Merge branch 'PHP-8.0'

* PHP-8.0:
  skip test with openssl < 1.1.0

Merge branch 'PHP-7.4' into PHP-8.0

* PHP-7.4:
  skip test with openssl < 1.1.0

skip test with openssl < 1.1.0

The test fails, but without any crash
(this test is designed to catch a crash)

Merge branch 'PHP-8.0'

* PHP-8.0:
  Capture Microsoft-defined HRESULT exit codes exit codes

Merge branch 'PHP-7.4' into PHP-8.0

* PHP-7.4:
  Capture Microsoft-defined HRESULT exit codes exit codes

Capture Microsoft-defined HRESULT exit codes exit codes

The lack of such a check leads to false-passes of tests on Windows
which expect no output, but produce a segfault or similar issue. I
discovered this a while ago due to bad tests in an extension I maintain.

Closes GH-6722.

Merge branch 'PHP-8.0'

* PHP-8.0:
  crc32 module, clang spurious warning removal.

crc32 module, clang spurious warning removal.

Closes GH-6725.

Speed up __sleep() and __wakeup() calls

Merge branch 'PHP-8.0'

* PHP-8.0:
  mysqlnd pam fix test error message

Merge branch 'PHP-7.4' into PHP-8.0

* PHP-7.4:
  mysqlnd pam fix test error message

mysqlnd pam fix test error message

Closes GH-6727.

Merge branch 'PHP-8.0'

* PHP-8.0:
  Fix potential file collision in dom tests
  Fix bug #80757 (Exit code is 0 when could not open file)
  Update NEWS

Merge branch 'PHP-7.4' into PHP-8.0

* PHP-7.4:
  Fix potential file collision in dom tests

Fix potential file collision in dom tests

Merge branch 'PHP-7.4' into PHP-8.0

Fix bug #80757 (Exit code is 0 when could not open file)

Update NEWS

Switch few functions useful in Symphony apps to new ZPP API.

Fixed error message

Allow pointer to end of memory in IS_UNSERIALIZED()

We already use <= for IS_SERIALIZED(), but the same general
problem can also occur for IS_UNSERIALIZED(). We don't seem to
hit this in practice prior to GH-5595 though.

Merge branch 'PHP-8.0'

* PHP-8.0:
  Fixed bug #80782 (DASM_S_RANGE_VREG on PHP_INT_MIN-1)

Fixed bug #80782 (DASM_S_RANGE_VREG on PHP_INT_MIN-1)

zend_compile.c: fix typo

Don't mind me, I just happened to be browsing this code. 🙃

Closes GH-6721.

[ci skip]

run-tests: add skip cache

Currently every --SKIPIF-- section in every test file results in 1
extra execution of PHP, every --EXTENSIONS-- section - in 2 executions.
This is quite wasteful, as skip checking code is extremely repetitive
and extensions are fixed for every binary/ini/command parameters
combination.

This patch adds caching to all such checks.

On my machine, the gains are quite noticeable: 36s instead of 43s
with -j16, 292s instead of 337s without concurrency. Cache stats are
3780 hits, 1247 misses in the latter case. In the future, tests could
be adjusted to have more uniform skip checks to improve performance even
more.

Closes GH-6681.

Merge branch 'PHP-8.0'

* PHP-8.0:
  run-tests: fixed exit code not being set on BORKED tests

Merge branch 'PHP-7.4' into PHP-8.0

* PHP-7.4:
  run-tests: fixed exit code not being set on BORKED tests

Merge branch 'PHP-7.3' into PHP-7.4

* PHP-7.3:
  run-tests: fixed exit code not being set on BORKED tests

run-tests: fixed exit code not being set on BORKED tests

When no test paths are specified this shows up when 'make test' is used on a PECL extension without specifying tests to run (or in php-src too, I guess...)

Closes GH-6717.

Fix Windows build

Don't add to a void pointer.

Merge branch 'PHP-8.0'

* PHP-8.0:
  Update php_version.h to proper version number

Merge branch 'PHP-7.4' into PHP-8.0

* PHP-7.4:
  Update php_version.h to proper version number

Update php_version.h to proper version number

Remove free_chunk API

This API only works in very limited circumstances -- relying on
it will invariably lead to leaks. The correct way to "free" something
from the pool is to use the checkpoint API.

Remove resize_chunk API

The last user has been dropped in the previous commit. This has
never worked meaningfully in the first place.

Fixed bug #80761

When row data split across multiple packets, allocate a temporary
buffer that can be reallocated, and only copy into the row buffer
pool arena once we know the final size. This avoids quadratic
memory usage for very large results.

Merge branch 'PHP-8.0'

* PHP-8.0:
  Fixed bug #80786

Fixed bug #80786

Don't use r0 as temporary register in math_double_long if it is
already used for a memory result.

This was already done in one branch, but not the other.

Fixed bug #80724

FilesystemIterator::FOLLOW_SYMLINKS is currently treated as a directory
key mode flag, even though it does not change the way that the key
during iteration is set. To address this, FOLLOW_SYMLINKS has been
converted into an OTHER flag.

Closes GH-6695.

Fix #80329: Add option to specify LOAD DATA LOCAL white list folder

 * allow the user to specify a folder where files that can be sent
   via LOAD DATA LOCAL can exist
 * add mysqli.local_infile_directory for mysqli
   (ignored if mysqli.allow_local_infile is enabled)
 * add PDO::MYSQL_ATTR_LOCAL_INFILE_DIRECTORY for pdo_mysql
   (ignored if PDO::MYSQL_ATTR_LOCAL_INFILE is enabled)
 * add related tests
 * fixes for building with libmysql 8.x
 * small improvement in existing tests
 * update php.ini-[development|production] files

Closes GH-6448.

Co-authored-by: Nikita Popov <nikic@php.net>

Merge branch 'PHP-8.0'

* PHP-8.0:
  Fix #75776: Flushing streams with compression filter is broken

Merge branch 'PHP-7.4' into PHP-8.0

* PHP-7.4:
  Fix #75776: Flushing streams with compression filter is broken

Fix #75776: Flushing streams with compression filter is broken

First, the `bzip2.compress` filter has the same issue as `zlib.deflate`
so we port the respective fix[1] to ext/bz2.

Second, there is still an issue, if a stream with an attached
compression filter is flushed before it is closed, without any writes
in between.  In that case, the compression is never finalized.  We fix
this by enforcing a `_php_stream_flush()` with the `closing` flag set
in `_php_stream_free()`, whenever a write filter is attached.  This
call is superfluous for most write filters, but does not hurt, even
when it is unnecessary.

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

Closes GH-6703.

Generate class entries from stubs for com, standard, xmlreader, xmlwriter, xsl, zip, Zend

Closes GH-6706

Merge branch 'PHP-8.0'

* PHP-8.0:
  Fix #80771: phpinfo(INFO_CREDITS) displays nothing in CLI

Merge branch 'PHP-7.4' into PHP-8.0

* PHP-7.4:
  Fix #80771: phpinfo(INFO_CREDITS) displays nothing in CLI

Fix #80771: phpinfo(INFO_CREDITS) displays nothing in CLI

There is no good reason not to show the credits in text based SAPIs,
except for brevity.  Thus, we suppress the credits from `php -i`.

Closes GH-6710.

Merge branch 'PHP-8.0'

* PHP-8.0:
  Fix #80774: session_name() problem with backslash

Merge branch 'PHP-7.4' into PHP-8.0

* PHP-7.4:
  Fix #80774: session_name() problem with backslash

Fix #80774: session_name() problem with backslash

Since we do no longer URL decode cookie names[1], we must not URL
encode the session name.  We need to prevent broken Set-Cookie headers,
by rejecting names which contain invalid characters.

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

Closes GH-6711.

Merge branch 'PHP-8.0'

* PHP-8.0:
  Fix trampoline leak on dynamic static call of non-static method

Fix trampoline leak on dynamic static call of non-static method

Fixes oss-fuzz #30317.

Don't resolve special class names

Adjust zend_resolve_class_name() to not resolve special class names.
This avoids the need to only call this function after a preliminary
check for non-default fetch types. Doing so is somewhat fragile
when dynamic class names are involved.

Fixes oss-fuzz #31139.

Merge branch 'PHP-8.0'

* PHP-8.0:
  Fixed bug #80781

Merge branch 'PHP-7.4' into PHP-8.0

* PHP-7.4:
  Fixed bug #80781

Fixed bug #80781

zend_find_array_dim_slow() may throw, make sure to handle this.
This backports the code we already use for this on PHP-8.0,
and also backports an exception check that makes this easier to
catch.

Fix typo [ci skip]

Closes GH-6713.

NEWS: Associate FR to xxHash additions

[ci skip]

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

UPGRADING: Add note about xxHash secret and fix a typo

[ci skip]

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

hash: Implement secret support for xxh3 and xxh128

A secret can be passed through the options array. The length is
currently in the range of 136 to 256 bytes. The concerned algos are
already marked as non serializable.

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

Revert "Remove class validation. zend_lookup_class_ex() performs it anyway."

This reverts commit 61cf1355fd53b6556f0502510ba3e1efbd04d242.

Class name validation is only performed if key is not passed. Here, lc_name is passed as key.

Remove class validation. zend_lookup_class_ex() performs it anyway.

Reduce ZPP API overhead

Fix closure GC handler for fake closures

Fixes oss-fuzz #31135.

Fixed NULL pointer dereference

Attempt to fix ext\standard\tests\file\basename_bug66395_variation2-win32.phpt and ext\standard\tests\file\pathinfo_basic1-win32.phpt

Improve basename(). Avoid calling mblen() for ASCII compatible locales.

Fix typo

Closes GH-6708

Signed-off-by: George Peter Banyard <girgias@php.net>

Fix crash during default value evaluation

Properly relove self::CONSTANT at compile time

Fixed bug #53826

We should handle the case where the method doesn't exist, and
where it is not visible the same way.

We don't need map_ptr slots for op_array.run_time_cache during preloading.

Generate ext/spl class entries from stubs

Closes GH-6709

Fixed bug #75474

For fake closures, we need to share static variables with the
original function, not work on a separate copy. Calling a function
through Closure::fromCallable() should have the same behavior as
calling it directly.

Fix static variable behavior with inheritance

When a method is inherited, the static variables will now always
use the initial values, rather than the values at the time of
inheritance. As such, behavior no longer depends on whether
inheritance happens before or after a method has been called.

This is implemented by always keeping static_variables as the
original values, and static_variables_ptr as the modified copy.

Closes GH-6705.

Initialize property to UNDEF on unserialize overwrite

The UNDEF marker here is important to prevent the creation of
a reference to the property currently being overwritten, which
would then leak.

This fixes oss-fuzz 6029559193534464, which was incorrectly
merged into oss-fuzz #30584 (which is reported at
https://github.com/google/oss-fuzz/issues/5211).

Merge branch 'PHP-8.0'

* PHP-8.0:
  Avoid signed integer overflow in substr()

Avoid signed integer overflow in substr()

Perform negation after the (size_t) cast rather than before,
so as to avoid a signed integer overflow for PHP_INT_MIN.

Fixes oss-fuzz #31069.

Don't use unmangled name if property not found

This restores the previous behavior for this case. We'll continue
to use the mangled name, even if it does not correspond to a
declared property.

This also fixes an assertion failure for the case of property
overwrite, as the add_new was not guaranteed to be "new" previously.

Fixes oss-fuzz #31045.

Merge branch 'PHP-8.0'

* PHP-8.0:
  Fix #80763: msgfmt_format() does not accept DateTime references

Merge branch 'PHP-7.4' into PHP-8.0

* PHP-7.4:
  Fix #80763: msgfmt_format() does not accept DateTime references

Fix #80763: msgfmt_format() does not accept DateTime references

`intl_zval_to_millis()` needs to cater to references.

Closes GH-6707.

Avoid writing zend_vm_opcodes.h if it did not change

Now that we track header changes, a change to zend_vm_opcodes.h
requires rebuilding essentially the whole tree. In most cases,
changes to zend_vm_def.h don't actually result in any header
changes though.

Avoid touching the file by first checking whether the to be written
contents are actually the same.

Coding style tweaks for zend_vm_gen.php

- fixed array style for $vm_kind_name and $op_types
- deleted unnecessary else statement for is_hot_helper function
- fixed concatenation style
- fixed style of using whitespace between "(" and switch/foreach - replaced !is_null with "!=="

Closes GH-6704.

Merge branch 'PHP-8.0'

* PHP-8.0:
  Color LEAK&FAIL like a failure

Color LEAK&FAIL like a failure

Merge branch 'PHP-8.0'

* PHP-8.0:
  Prep for 8.0.4

Prep for 8.0.4

unserialize() optimization. Omit class name validation before hash lookup, and perform it only before autoloading.

Merge branch 'PHP-8.0'

* PHP-8.0:
  Fixed bug #80745 (JIT produces Assert failure and UNKNOWN:0 var_dumps in code involving bitshifts)

Fixed bug #80745 (JIT produces Assert failure and UNKNOWN:0 var_dumps in code involving bitshifts)

Merge branch 'PHP-8.0'

* PHP-8.0:
  Remove generated zend_jit_x86.c on `make clean`

Remove generated zend_jit_x86.c on `make clean`

Not removing this causes build failure when reconfiguring and rebuilding after a `make clean`, e.g. enabling/disabling ZTS.

This makes https://bugs.php.net/bug.php?id=80561 more bearable. Ideally it would be rebuilt automatically on configuration change if necessary, but I have no idea how to implement this.

Closes GH-6702.