| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
| |
Closes GH-6173
|
|
|
|
|
|
| |
This is an annoying edge case that regularly gets broken. As we're
not aware of significant users of this API, and there are other
ways to hook this, remove support for EXT_NOP.
|
|
|
|
| |
Closes GH-6049.
|
|
|
|
|
|
| |
We might just want to drop this completely, but at least don't
enable it by default. It already gets disabled by a number of
SAPIs, but we should make that the default state.
|
| |
|
|
|
|
| |
Closes GH-5758
|
|
|
|
|
|
| |
getrusage supports only two fields. The network api sits in the network lib.
Closes GH-5732.
|
|
|
|
| |
Closes GH-5676.
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
We fix (hopefully) all instances of:
* <https://docs.microsoft.com/en-us/cpp/error-messages/compiler-warnings/compiler-warning-level-1-c4005>
* <https://docs.microsoft.com/en-us/cpp/error-messages/compiler-warnings/compiler-warning-level-1-c4024>
* <https://docs.microsoft.com/en-us/cpp/error-messages/compiler-warnings/compiler-warning-level-1-c4028>
* <https://docs.microsoft.com/en-us/cpp/error-messages/compiler-warnings/compiler-warning-level-1-c4047>
* <https://docs.microsoft.com/en-us/cpp/error-messages/compiler-warnings/compiler-warning-level-1-c4087>
* <https://docs.microsoft.com/en-us/cpp/error-messages/compiler-warnings/compiler-warning-level-1-c4090>
* <https://docs.microsoft.com/en-us/cpp/error-messages/compiler-warnings/compiler-warning-level-1-c4273>
* <https://docs.microsoft.com/en-us/cpp/error-messages/compiler-warnings/compiler-warning-level-1-c4312>
`zend_llist_add_element()` and `zend_llist_prepend_element()` now
explicitly expect a *const* pointer.
We use the macro `ZEND_VOIDP()` instead of a `(void*)` cast to suppress
C4090; this should prevent accidential removal of the cast by
clarifying the intention, and makes it easier to remove the casts if
the issue[1] will be resolved sometime.
[1] <https://developercommunity.visualstudio.com/content/problem/390711/c-compiler-incorrect-propagation-of-const-qualifie.html>
|
|\
| |
| |
| |
| |
| |
| |
| | |
* PHP-7.4:
Enable phpdbg tests on AppVeyor
Make phpdbg test portable
Fix several mostly Windows related phpdbg bugs
Fix #73926: phpdbg will not accept input on restart execution
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
* Properly initialize PHPDBG_G(watch_tmp)
Otherwise that may cause segfaults in ZTS builds.
* Deactivate potentially remaining watchpoints after REPL
Otherwise the memory could still be protected, resulting in segfaults
during shutdown.
* NULL zend_handlers_table after freeing
As of commit 4130fe4[1], the `zend_handlers_table` is explicitly
freed in the `zend_vm_dtor()`. Since phpdbg (and maybe some other
SAPIs) may restart the engine afterwards, we have to make sure that
the table is also NULLed.
* Only set context option if there is a context
In other words, we must not follow the null pointer.
* Cater to file handles without attached console
File handles do not necessarily have an attached console (for
instance, pipes do not), in which case `GetConsoleScreenBufferInfo()`
fails. In this case we set a default value (`40`) for lines like on
other systems.
[1] <http://git.php.net/?p=php-src.git;a=commit;h=4130fe437a5db7ead1444d3748bd0fbad9829cb2>
|
|\ \
| |/
| |
| |
| | |
* PHP-7.4:
Fix INI setting member name
|
| |\
| | |
| | |
| | |
| | | |
* PHP-7.3:
Fix INI setting member name
|
| | |
| | |
| | |
| | |
| | |
| | | |
Since the member is not used in `OnUpdateEol()` that's not really an
issue, but still it's confusing to apparently have two INI settings
targeting the same member.
|
| | |
| | |
| | |
| | |
| | | |
For apache2handler, fpm, litespeed, phpdbg, specifically.
Partially implements GH-5295
|
| | | |
|
| | |
| | |
| | |
| | | |
Closes GH-5404
|
| | | |
|
| | | |
|
| | |
| | |
| | |
| | | |
Closes GH-4867.
|
| | |
| | |
| | |
| | | |
Add a zpp_none() call, convert E_ERROR to ValueError.
|
|/ /
| |
| |
| | |
Closes GH-4732.
|
| |
| |
| |
| |
| | |
The meaning of the limit changed in 7.4, it now points one past the
end. Adjust code accordingly.
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
The php_stream_read() and php_stream_write() functions now return
an ssize_t value, with negative results indicating failure. Functions
like fread() and fwrite() will return false in that case.
As a special case, EWOULDBLOCK and EAGAIN on non-blocking streams
should not be regarded as error conditions, and be reported as
successful zero-length reads/writes instead. The handling of EINTR
remains unclear and is internally inconsistent (e.g. some code-paths
will automatically retry on EINTR, while some won't).
I'm landing this now to make sure the stream wrapper ops API changes
make it into 7.4 -- however, if the user-facing changes turn out to
be problematic we have the option of clamping negative returns to
zero in php_stream_read() and php_stream_write() to restore the
old behavior in a relatively non-intrusive manner.
|
| | |
|
| |
| |
| |
| |
| |
| | |
Instead of guessing what the address of the first arena allocation
is going to be, embed the sentinel in the oplog_list structure
directly.
|
| | |
|
| | |
|
| |
| |
| |
| |
| |
| |
| |
| | |
In particular, make sure that everything using zmm is released
before zmm is shut down. phpdbg currently gets away with this,
because either a) its custom handlers are used and no auto-free
happens or b) the system allocator is used and no auto-free happens.
With the tracking allocator for asan this no longer works.
|
| |
| |
| |
| |
| | |
Uses system allocator when backing up settings ... let's avoid
unnecessary confusion.
|
| | |
|
| |
| |
| |
| |
| | |
We want the class declaration to be available while compiling the
parent class.
|
| | |
|
| | |
|
| | |
|
| | |
|
| |
| |
| |
| |
| |
| | |
This patch follows previous license year ranges updates. With new
approach source code files now have simplified headers with license
information without year ranges.
|
| | |
|
| |
| |
| |
| | |
released in ZTS build, and this leads to crash.
|
| |
| |
| |
| |
| |
| |
| | |
zend_class_entry->interface_names.
Move interface implementation code into ZEND_DECLARE_*CLASS opcodes.
Remove ZEND_ADD_INTERFACE and ZEND_VERIFY_ABSTRACT_CLASS opcodes.
|
|/
|
|
|
| |
Move trait binding code into ZEND_DECLARE_*CLASS opcodes.
Remove ZEND_ADD_TRIAIT and ZEND_BIND_TRAITS opcodes.
|
|
|
|
| |
inside zend_constant.value.
|
| |
|
|\
| |
| |
| |
| | |
* PHP-7.2:
Fix arginfo wrt. optional/required parameters
|
| |\
| | |
| | |
| | |
| | | |
* PHP-7.1:
Fix arginfo wrt. optional/required parameters
|
| | |
| | |
| | |
| | |
| | | |
All parameters of phpdbg_color(), phpdbg_exec() and phpdbg_prompt() are
required. We mark them as such.
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
The sole purpose of `PHPDBG_FILE`, `PHPDBG_METHOD`, `PHPDBG_LINENO` and
`PHPDBG_FUNC` has been to be passed as first argument to `phpdbg_break`.
However, this functions is replaced as of PHP 5.6.3 by
`phpdbg_break_file`, `phpdbg_break_method` and 'phpdbg_break_func`,
respectively. Therefore, we're finally removing the useless constants.
|
|\ \ \
| |/ /
| | |
| | |
| | | |
* PHP-7.2:
Allocate default ini values into persistent memory
|
| |\ \
| | |/
| | |
| | |
| | | |
* PHP-7.1:
Allocate default ini values into persistent memory
|
| | | |
|