| Commit message (Collapse) | Author | Age | Files | Lines |
... | |
| | | |
|
| | | |
|
| | |
| | |
| | |
| | |
| | | |
Make sure the string key is not released while throwing the
undefined index warning.
|
| | |
| | |
| | |
| | |
| | | |
The fix here is essentially the same as for bug #78598, just for
the undefined variable notice, rather than the undefined index one.
|
|\ \ \
| |/ /
| | |
| | |
| | | |
* PHP-7.3:
Don't use deprecated curly brace offset syntax
|
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
(cherry picked from commit 7ec3aa1871074f5de25865af42c984a7668eb85f)
Better safe than sorry in case someone ever builds PHP 7.3 with a
future version of PHP SDK with bundled PHP 8.
|
| | | |
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
When performing an RW modification of an array offset, the undefined
offset warning may call an error handler / OB callback, which may
destroy the array we're supposed to change. Detect this by temporarily
incrementing the reference count. If we find that the array has been
modified/destroyed in the meantime, we do nothing -- the execution
model here would be that the modification has happened on the destroyed
version of the array.
|
|\ \ \
| |/ /
| | |
| | |
| | | |
* PHP-7.3:
Fixed bug #79792
|
| | |
| | |
| | |
| | |
| | |
| | | |
We need to remove the iterators even if the array is empty (we
will not create one if the first place, but the array may become
empty after the fact).
|
| | |
| | |
| | |
| | | |
ASSIGN_OBJ_REF was not handling in zend_wrong_string_offset.
|
|\ \ \
| |/ /
| | |
| | |
| | | |
* PHP-7.3:
Fixed bug #79778
|
| | |
| | |
| | |
| | |
| | |
| | | |
In the interest of avoiding side-effects during dumping, I'm
replacing the value with a <constant ast> string instead of
performing an update constant operation.
|
| | |
| | |
| | |
| | |
| | | |
Make sure we don't drop the by-reference check when passing the
result of a VM builtin function.
|
| | |
| | |
| | |
| | |
| | | |
`st_dev` deliberately overflows on such systems, cf.
<http://svn.php.net/viewvc?view=revision&revision=350100>.
|
| | | |
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
We modify _basic1.phpt so it runs on Windows as well. The other test
cases hit the issue that `readlink()` fails normally for regular files,
but succeeds on Windows[1]. Therefore, we split these tests, but still
fix the skip reasons.
[1] <http://svn.php.net/viewvc?view=revision&revision=350097>
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Most of these have been skipped on Windows for no good reason (`lstat`
is available there as of PHP 4). Several others would only fail,
because the `blksize` and `blocks` elements are always `-1` on Windows,
which can easily be fixed by using `%i` format specifiers instead of
`%d`.
|
| | |
| | |
| | |
| | | |
That has apparently been overlooked.
|
| | | |
|
| | |
| | |
| | |
| | |
| | |
| | | |
These options may have the same value as options at other levels.
This issue showed up on ppc64el.
|
| | |
| | |
| | |
| | | |
Apparently this is EDEADLOCK on some systems.
|
|\ \ \
| |/ /
| | |
| | |
| | | |
* PHP-7.3:
Remove bogus generator iterator dtor
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Fixes a use-after-free encountered in Symfony's SecurityBundle.
I don't have a reproducer for this, and believe the issue can only
occur if we leak an iterator (the leak is a separate issue).
We should not free the generator iterator here, because we do not
own it. The code that fetched the iterator is responsible for
releasing it. In the rare case where we do hit this code-path,
we cause a use-after-free.
|
|\ \ \
| |/ /
| | |
| | |
| | | |
* PHP-7.3:
Fix leak when setting cyclic previous exception in finally
|
| | |
| | |
| | |
| | | |
A curious exception handling pattern found in Symfony's HttpClient.
|
|\ \ \
| |/ /
| | |
| | |
| | | |
* PHP-7.3:
Fix #70362: Can't copy() large 'data://' with open_basedir
|
| | |
| | |
| | |
| | |
| | | |
open_basedir is only relevant for plain files, so there is no need to
check it for other URL wrappers.
|
|\ \ \
| |/ /
| | |
| | |
| | | |
* PHP-7.3:
Fix #63208: BSTR to PHP string conversion not binary safe
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
A `BSTR` is similar to a `zend_string`; it stores the length of the
string just before the actual string, and thus the string may contain
NUL bytes. However, `php_com_olestring_to_string()` is supposed to
deal with arbitrary `OLECHAR*`s which may not be `BSTR`s, so we
introduce `php_com_bstr_to_string()` and use it for the only case where
we actually have to deal with `BSTR`s which may contain NUL bytes.
Contrary to `php_com_olestring_to_string()` we return a `zend_string`,
so we can save the re-allocation when converting to a `zval`.
We also cater to `php_com_string_to_olestring()` not being binary safe,
with basically the same fix we did for `php_com_olestring_to_string()`.
|
|\ \ \
| |/ /
| | |
| | |
| | | |
* PHP-7.3:
Fix #79756: finfo_file crash (FILEINFO_MIME)
|
| | |
| | |
| | |
| | |
| | | |
If `ctime` or `asctime` return `NULL`, we must not attempt to copy the
buffer, but rather return `NULL` as well.
|
| | |
| | |
| | |
| | | |
Fixes the failure in bug79740.phpt with opcache.
|
|\ \ \
| |/ /
| | |
| | |
| | | |
* PHP-7.3:
Fixed bug #79741
|
| | | |
|
| | | |
|
|\ \ \
| |/ /
| | |
| | |
| | | |
* PHP-7.3:
Fixed bug #79030 Use usec from apache request time
|
| | |
| | |
| | |
| | |
| | |
| | | |
Don't unnecessarily truncate to milliseconds.
Closes GH-5760.
|
| | |
| | |
| | |
| | |
| | | |
Avoid throwing a DES salt deprecation warning if the libc crypt
implementation is used.
|
| | |
| | |
| | |
| | | |
We have to unlock the environment before bailing out.
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
`ReflectionClass` allows reading of the values of private and protected
constants, and also to get private and protected static methods.
Therefore getting the values of private and protected static properties
is also permissible, especially since `::getStaticProperties()` already
allows to do so.
We also allow ::setStaticPropertyValue() to modify private and
protected properties, because otherwise this method is useless, as
modifying public properties can be done directly.
|
| | |
| | |
| | |
| | |
| | | |
When retrieving the static class properties via reflection, we have to
cater to possible modifications.
|
| | |
| | |
| | |
| | | |
This reverts commit a895bb6885fbceea3e8375816969d5510d8d082e.
|
| | |
| | |
| | |
| | |
| | | |
When retrieving the static class properties via reflection, we have to
cater to possible modifications.
|
|\ \ \
| |/ /
| | |
| | |
| | | |
* PHP-7.3:
Fix #55857: ftp_size on large files
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
`atol()` returns a `long` which is not the same as `zend_long` on
LLP64; we use `ZEND_ATOL()` instead.
There is no need for a new test case, since filesize_large.phpt already
tests for that behavior; unfortunately, the FTP test suite relies on
`pcntl_fork()` and therefore cannot be run on Windows.
|
| | |
| | |
| | |
| | | |
Still fails intermittently.
|
|\ \ \
| |/ /
| | |
| | |
| | | |
* PHP-7.3:
7.3 is now 7.3.21-dev
|
| | | |
|
| | | |
|