Skip to content

Commit 20ef51d

Browse files
dstogovdstogov
committed
Fixed bug #78937 (Preloading unlinkable anonymous class can segfault)
1 parent 518a8f8 commit 20ef51d

File tree

10 files changed

+162
-2
lines changed

10 files changed

+162
-2
lines changed

‎Zend/zend_compile.c‎

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1068,8 +1068,12 @@ ZEND_API int do_bind_class(zval *lcname, zend_string *lc_parent_name) /* {{{ */
10681068

10691069
if (UNEXPECTED(!zv)) {
10701070
ce = zend_hash_find_ptr(EG(class_table), Z_STR_P(lcname));
1071-
ZEND_ASSERT(ce && "Class with lcname should be registered");
1072-
zend_error_noreturn(E_COMPILE_ERROR, "Cannot declare %s %s, because the name is already in use", zend_get_object_type(ce), ZSTR_VAL(ce->name));
1071+
if (ce) {
1072+
zend_error_noreturn(E_COMPILE_ERROR, "Cannot declare %s %s, because the name is already in use", zend_get_object_type(ce), ZSTR_VAL(ce->name));
1073+
} else {
1074+
ZEND_ASSERT(EG(current_execute_data)->func->op_array.fn_flags & ZEND_ACC_PRELOADED);
1075+
zend_error_noreturn(E_ERROR, "Class %s wasn't preloaded", Z_STRVAL_P(lcname));
1076+
}
10731077
return FAILURE;
10741078
}
10751079

‎Zend/zend_vm_def.h‎

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -7306,6 +7306,11 @@ ZEND_VM_HANDLER(146, ZEND_DECLARE_ANON_CLASS, ANY, ANY, CACHE_SLOT)
73067306
if (UNEXPECTED(ce == NULL)) {
73077307
zend_string *rtd_key = Z_STR_P(RT_CONSTANT(opline, opline->op1));
73087308
zv = zend_hash_find_ex(EG(class_table), rtd_key, 1);
7309+
if (UNEXPECTED(zv == NULL)) {
7310+
SAVE_OPLINE();
7311+
ZEND_ASSERT(EX(func)->op_array.fn_flags & ZEND_ACC_PRELOADED);
7312+
zend_error_noreturn(E_ERROR, "Anonymous class wasn't preloaded");
7313+
}
73097314
ZEND_ASSERT(zv != NULL);
73107315
ce = Z_CE_P(zv);
73117316
if (!(ce->ce_flags & ZEND_ACC_LINKED)) {

‎Zend/zend_vm_execute.h‎

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2446,6 +2446,11 @@ static ZEND_OPCODE_HANDLER_RET ZEND_FASTCALL ZEND_DECLARE_ANON_CLASS_SPEC_HANDLE
24462446
if (UNEXPECTED(ce == NULL)) {
24472447
zend_string *rtd_key = Z_STR_P(RT_CONSTANT(opline, opline->op1));
24482448
zv = zend_hash_find_ex(EG(class_table), rtd_key, 1);
2449+
if (UNEXPECTED(zv == NULL)) {
2450+
SAVE_OPLINE();
2451+
ZEND_ASSERT(EX(func)->op_array.fn_flags & ZEND_ACC_PRELOADED);
2452+
zend_error_noreturn(E_ERROR, "Anonymous class wasn't preloaded");
2453+
}
24492454
ZEND_ASSERT(zv != NULL);
24502455
ce = Z_CE_P(zv);
24512456
if (!(ce->ce_flags & ZEND_ACC_LINKED)) {

‎ext/opcache/tests/bug78937_1.phpt‎

Lines changed: 22 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,22 @@
1+
--TEST--
2+
Bug #78937.1 (Preloading unlinkable anonymous class can segfault)
3+
--INI--
4+
opcache.enable=1
5+
opcache.enable_cli=1
6+
opcache.optimization_level=-1
7+
opcache.preload={PWD}/preload_bug78937.inc
8+
--SKIPIF--
9+
<?php require_once('skipif.inc'); ?>
10+
--FILE--
11+
<?php
12+
class Bar {
13+
}
14+
var_dump(foo());
15+
?>
16+
--EXPECTF--
17+
Warning: Can't preload unlinked class Foo: Unknown parent Bar in %spreload_bug78937.inc on line 6
18+
19+
Warning: Can't preload unlinked class class@anonymous: Unknown parent Bar in %spreload_bug78937.inc on line 3
20+
21+
Fatal error: Anonymous class wasn't preloaded in %spreload_bug78937.inc on line 3
22+

‎ext/opcache/tests/bug78937_2.phpt‎

Lines changed: 22 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,22 @@
1+
--TEST--
2+
Bug #78937.2 (Preloading unlinkable anonymous class can segfault)
3+
--INI--
4+
opcache.enable=1
5+
opcache.enable_cli=1
6+
opcache.optimization_level=-1
7+
opcache.preload={PWD}/preload_bug78937.inc
8+
--SKIPIF--
9+
<?php require_once('skipif.inc'); ?>
10+
--FILE--
11+
<?php
12+
include(__DIR__ . "/preload_bug78937.inc");
13+
class Bar {
14+
}
15+
var_dump(foo());
16+
?>
17+
--EXPECTF--
18+
Warning: Can't preload unlinked class Foo: Unknown parent Bar in %spreload_bug78937.inc on line 6
19+
20+
Warning: Can't preload unlinked class class@anonymous: Unknown parent Bar in %spreload_bug78937.inc on line 3
21+
object(class@anonymous)#%d (0) {
22+
}

‎ext/opcache/tests/bug78937_3.phpt‎

Lines changed: 24 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,24 @@
1+
--TEST--
2+
Bug #78937.3 (Preloading unlinkable anonymous class can segfault)
3+
--INI--
4+
opcache.enable=1
5+
opcache.enable_cli=1
6+
opcache.optimization_level=-1
7+
opcache.preload={PWD}/preload_bug78937.inc
8+
--SKIPIF--
9+
<?php require_once('skipif.inc'); ?>
10+
--FILE--
11+
<?php
12+
include(__DIR__ . "/preload_bug78937.inc");
13+
var_dump(foo());
14+
?>
15+
--EXPECTF--
16+
Warning: Can't preload unlinked class Foo: Unknown parent Bar in %spreload_bug78937.inc on line 6
17+
18+
Warning: Can't preload unlinked class class@anonymous: Unknown parent Bar in %spreload_bug78937.inc on line 3
19+
20+
Fatal error: Uncaught Error: Class 'Bar' not found in %spreload_bug78937.inc:3
21+
Stack trace:
22+
#0 %sbug78937_3.php(3): foo()
23+
#1 {main}
24+
thrown in %spreload_bug78937.inc on line 3

‎ext/opcache/tests/bug78937_4.phpt‎

Lines changed: 22 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,22 @@
1+
--TEST--
2+
Bug #78937.4 (Preloading unlinkable anonymous class can segfault)
3+
--INI--
4+
opcache.enable=1
5+
opcache.enable_cli=1
6+
opcache.optimization_level=-1
7+
opcache.preload={PWD}/preload_bug78937.inc
8+
--SKIPIF--
9+
<?php require_once('skipif.inc'); ?>
10+
--FILE--
11+
<?php
12+
class Bar {
13+
}
14+
bar();
15+
var_dump(new Foo);
16+
?>
17+
--EXPECTF--
18+
Warning: Can't preload unlinked class Foo: Unknown parent Bar in %spreload_bug78937.inc on line 6
19+
20+
Warning: Can't preload unlinked class class@anonymous: Unknown parent Bar in %spreload_bug78937.inc on line 3
21+
22+
Fatal error: Class foo wasn't preloaded in %spreload_bug78937.inc on line 6

‎ext/opcache/tests/bug78937_5.phpt‎

Lines changed: 23 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,23 @@
1+
--TEST--
2+
Bug #78937.5 (Preloading unlinkable anonymous class can segfault)
3+
--INI--
4+
opcache.enable=1
5+
opcache.enable_cli=1
6+
opcache.optimization_level=-1
7+
opcache.preload={PWD}/preload_bug78937.inc
8+
--SKIPIF--
9+
<?php require_once('skipif.inc'); ?>
10+
--FILE--
11+
<?php
12+
include(__DIR__ . "/preload_bug78937.inc");
13+
class Bar {
14+
}
15+
bar();
16+
var_dump(new Foo);
17+
?>
18+
--EXPECTF--
19+
Warning: Can't preload unlinked class Foo: Unknown parent Bar in %spreload_bug78937.inc on line 6
20+
21+
Warning: Can't preload unlinked class class@anonymous: Unknown parent Bar in %spreload_bug78937.inc on line 3
22+
object(Foo)#%d (0) {
23+
}

‎ext/opcache/tests/bug78937_6.phpt‎

Lines changed: 25 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,25 @@
1+
--TEST--
2+
Bug #78937.6 (Preloading unlinkable anonymous class can segfault)
3+
--INI--
4+
opcache.enable=1
5+
opcache.enable_cli=1
6+
opcache.optimization_level=-1
7+
opcache.preload={PWD}/preload_bug78937.inc
8+
--SKIPIF--
9+
<?php require_once('skipif.inc'); ?>
10+
--FILE--
11+
<?php
12+
include(__DIR__ . "/preload_bug78937.inc");
13+
bar();
14+
var_dump(new Foo);
15+
?>
16+
--EXPECTF--
17+
Warning: Can't preload unlinked class Foo: Unknown parent Bar in %spreload_bug78937.inc on line 6
18+
19+
Warning: Can't preload unlinked class class@anonymous: Unknown parent Bar in %spreload_bug78937.inc on line 3
20+
21+
Fatal error: Uncaught Error: Class 'Bar' not found in %spreload_bug78937.inc:6
22+
Stack trace:
23+
#0 %sbug78937_6.php(3): bar()
24+
#1 {main}
25+
thrown in %spreload_bug78937.inc on line 6

‎ext/opcache/tests/preload_bug78937.inc‎

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,8 @@
1+
<?php
2+
function foo() {
3+
return new class extends Bar {};
4+
}
5+
function bar() {
6+
class Foo extends Bar {
7+
}
8+
}

0 commit comments

Comments
 (0)