diff options
| author | Christoph M. Becker <cmbecker69@gmx.de> | 2020-09-22 16:11:56 +0200 |
|---|---|---|
| committer | Christoph M. Becker <cmbecker69@gmx.de> | 2020-09-22 16:12:37 +0200 |
| commit | ff0f6c26c20b97e47f2bb70f631165e3a8801aac (patch) | |
| tree | 952e64439718291568b79324740488e0a70ec97b /main/streams | |
| parent | f445e9cb93eff1856748aaa6d020b74a653d2d76 (diff) | |
| parent | 5ed0602ec622274bf5672304ae414e5ecb2e5167 (diff) | |
| download | php-git-ff0f6c26c20b97e47f2bb70f631165e3a8801aac.tar.gz | |
Merge branch 'PHP-7.3' into PHP-7.4
* PHP-7.3:
Fix #76943: Inconsistent stream_wrapper_restore() errors
Diffstat (limited to 'main/streams')
| -rw-r--r-- | main/streams/userspace.c | 13 |
1 files changed, 7 insertions, 6 deletions
diff --git a/main/streams/userspace.c b/main/streams/userspace.c index 9cf1580a35..cf8f59b756 100644 --- a/main/streams/userspace.c +++ b/main/streams/userspace.c @@ -559,23 +559,24 @@ PHP_FUNCTION(stream_wrapper_restore) { zend_string *protocol; php_stream_wrapper *wrapper; - HashTable *global_wrapper_hash; + HashTable *global_wrapper_hash, *wrapper_hash; if (zend_parse_parameters(ZEND_NUM_ARGS(), "S", &protocol) == FAILURE) { RETURN_FALSE; } global_wrapper_hash = php_stream_get_url_stream_wrappers_hash_global(); - if (php_stream_get_url_stream_wrappers_hash() == global_wrapper_hash) { - php_error_docref(NULL, E_NOTICE, "%s:// was never changed, nothing to restore", ZSTR_VAL(protocol)); - RETURN_TRUE; - } - if ((wrapper = zend_hash_find_ptr(global_wrapper_hash, protocol)) == NULL) { php_error_docref(NULL, E_WARNING, "%s:// never existed, nothing to restore", ZSTR_VAL(protocol)); RETURN_FALSE; } + wrapper_hash = php_stream_get_url_stream_wrappers_hash(); + if (wrapper_hash == global_wrapper_hash || zend_hash_find_ptr(wrapper_hash, protocol) == wrapper) { + php_error_docref(NULL, E_NOTICE, "%s:// was never changed, nothing to restore", ZSTR_VAL(protocol)); + RETURN_TRUE; + } + /* A failure here could be okay given that the protocol might have been merely unregistered */ php_unregister_url_stream_wrapper_volatile(protocol); |