Use the passed 'this' pointer instead of ZEND_THIS (#8854)

Removes the horrible hack usage of defining execute_data to be able to use ZEND_THING

Refactor spl_filesystem_object_get_path() to return zend_string*

Remove Z_SPLFILESYSTEM_P macro

Remove one level of macro expansion hell

Move some private SPL Directory elements out of the header

Early return in spl_filesystem_file_rewind()

Use bool for silent parameter in SPL directory.c

Prevent fclose on underlying SplFileObject file stream. (#7920)

Fix GH-8563 Different results for seek() on SplFileObject and SplTempFileObject

With memory streams if we get a NULL buffer we must not instantiate an empty line

Remove internal usage of SplFileInfo::_bad_state_ex() method (#8318)

* Use standard VM handling instead
* Deprecate the method as it is now useless

Fix GH-8273: SplFileObject: key() returns wrong value

Fix GH-8273: SplFileObject: key() returns wrong value

Refactor SplFileObject CSV methods (#8317)

Use early returns, which fixes ValueError order of arguments and flattens the code

Add missing ZPP checks in SPL Directory (#8325)

Co-authored-by: Máté Kocsis <kocsismate@woohoolabs.com>

Minor refactoring of spl_directory.c

Using more appropriate types
Fix comments
Early returns

Replace memcmp() with zend_string functions (#8216)

* ext/oci8: use zend_string_equals()

Eliminate duplicate code.

* main/php_variables: use zend_string_equals_literal()

Replace memcmp() with zend_string functions (#8216)

* ext/oci8: use zend_string_equals()

Eliminate duplicate code.

* main/php_variables: use zend_string_equals_literal()

Eliminate duplicate code.

* Zend/zend_string: add zend_string_equals_cstr()

Allows eliminating duplicate code.

* Zend, ext/{opcache,standard}, main/output: use zend_string_equals_cstr()

Eliminate duplicate code.

* Zend/zend_string: add zend_string_starts_with()

* ext/{opcache,phar,spl,standard}: use zend_string_starts_with()

This adds missing length checks to several callers, e.g. in
cache_script_in_shared_memory(). This is important when the
zend_string is shorter than the string parameter, when memcmp()
happens to check backwards; this can result in an out-of-bounds memory
access.

show more ...

Fix GH-8121: SplFileObject - seek and key with csv file inconsistent

First, we must not free the current line before we call
`spl_filesystem_file_read_csv()`, because then the `current_l

Fix GH-8121: SplFileObject - seek and key with csv file inconsistent

First, we must not free the current line before we call
`spl_filesystem_file_read_csv()`, because then the `current_line` will
not be properly updated. Since the EOF check is superfluous here, we
move that part of the code to the branch for subtypes. This issue has
been introduced by the fix for bug 75917.

Second, we only must increase the `current_line` if we're not reading
ahead. This issue has been introduced by the fix for bug 62004.

Closes GH-8138.

show more ...

Fix GH-7809: Cloning a faked SplFileInfo object may segfault

While the `path` is not supposed to be `NULL` for normal operation, it
is possible to create `SplFileInfo` objects where that

Fix GH-7809: Cloning a faked SplFileInfo object may segfault

While the `path` is not supposed to be `NULL` for normal operation, it
is possible to create `SplFileInfo` objects where that is the case, and
we must not follow the null pointer.

Closes GH-7814.

show more ...

Fix #75917: SplFileObject::seek broken with CSV flags

Closes GH-7697.

Use more compact representation for packed arrays.

- for packed arrays we store just an array of zvals without keys.
- the elements of packed array are accessible throuf as ht->arPacked[

Use more compact representation for packed arrays.

- for packed arrays we store just an array of zvals without keys.
- the elements of packed array are accessible throuf as ht->arPacked[i]
instead of ht->arData[i]
- in addition to general ZEND_HASH_FOREACH_* macros, we introduced similar
familied for packed (ZEND_HASH_PACKED_FORECH_*) and real hashes
(ZEND_HASH_MAP_FOREACH_*)
- introduced an additional family of macros to access elements of array
(packed or real hashes) ZEND_ARRAY_ELEMET_SIZE, ZEND_ARRAY_ELEMET_EX,
ZEND_ARRAY_ELEMET, ZEND_ARRAY_NEXT_ELEMENT, ZEND_ARRAY_PREV_ELEMENT
- zend_hash_minmax() prototype was changed to compare only values

Because of smaller data set, this patch may show performance improvement
on some apps and benchmarks that use packed arrays. (~1% on PHP-Parser)

TODO:
- sapi/phpdbg needs special support for packed arrays (WATCH_ON_BUCKET).
- zend_hash_sort_ex() may require converting packed arrays to hash.

show more ...

Fix #81477: LimitIterator + SplFileObject regression in 8.0.1

We must not free the read line, if the `READ_AHEAD` flag is set. This
also restores the expectations of SplFileObject_next_

Fix #81477: LimitIterator + SplFileObject regression in 8.0.1

We must not free the read line, if the `READ_AHEAD` flag is set. This
also restores the expectations of SplFileObject_next_variation002.phpt.

Closes GH-7518.

show more ...

Fix FileSystemIterator without SKIP_DOTS

There were two separate bugs here:
* The get_iterator implementation did not match the Iterator
implementation. In particular, get_iterat

Fix FileSystemIterator without SKIP_DOTS

There were two separate bugs here:
* The get_iterator implementation did not match the Iterator
implementation. In particular, get_iterator did not respect
SKIP_DOTS.
* The constructor did not honor an explicitly omitted SKIP_DOTS
flag. It could still be unset through setFlags() though.

show more ...

Fix typo [no-ci]

Fix SplFileObject::fseek() method description (#7321)

Add ZEND_ACC_NOT_SERIALIZABLE flag

This prevents serialization and unserialization of a class and its
children in a way that does not depend on the zend_class_serialize_deny
and zend

Add ZEND_ACC_NOT_SERIALIZABLE flag

This prevents serialization and unserialization of a class and its
children in a way that does not depend on the zend_class_serialize_deny
and zend_class_unserialize_deny handlers that will be going away
in PHP 9 together with the Serializable interface.

In stubs, `@not-serializable` can be used to set this flag.

This patch only uses the new flag for a handful of Zend classes,
converting the remainder is left for later.

Closes GH-7249.
Fixes bug #81111.

show more ...

Declare tentative return types for ext/spl - part 1 (#7115)