Skip to content

Commit 9792f0d

Browse files
committed
JIT: Fix type store
Fixes oss-fuzz #45190
1 parent 2b2aeb9 commit 9792f0d

File tree

2 files changed

+28
-8
lines changed

2 files changed

+28
-8
lines changed

ext/opcache/jit/zend_jit_trace.c

Lines changed: 6 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -4642,18 +4642,16 @@ static const void *zend_jit_trace(zend_jit_trace_rec *trace_buffer, uint32_t par
46424642
}
46434643
CHECK_OP2_TRACE_TYPE();
46444644
op1_info = OP1_INFO();
4645-
if ((op1_info & (MAY_BE_ANY|MAY_BE_UNDEF|MAY_BE_GUARD)) == MAY_BE_LONG
4646-
|| (op1_info & (MAY_BE_ANY|MAY_BE_UNDEF|MAY_BE_GUARD)) == MAY_BE_DOUBLE) {
4647-
if (STACK_MEM_TYPE(stack, EX_VAR_TO_NUM(opline->op1.var)) != IS_LONG
4648-
&& STACK_MEM_TYPE(stack, EX_VAR_TO_NUM(opline->op1.var)) != IS_DOUBLE) {
4649-
/* type may be not set */
4650-
op1_info |= MAY_BE_NULL;
4651-
}
4652-
}
46534645
CHECK_OP1_TRACE_TYPE();
46544646
op1_def_info = OP1_DEF_INFO();
46554647
op1_addr = OP1_REG_ADDR();
46564648
op1_def_addr = OP1_DEF_REG_ADDR();
4649+
if (Z_MODE(op1_def_addr) != IS_REG &&
4650+
STACK_TYPE(stack, EX_VAR_TO_NUM(opline->op1.var)) !=
4651+
STACK_MEM_TYPE(stack, EX_VAR_TO_NUM(opline->op1.var))) {
4652+
/* type may be not set */
4653+
op1_info |= MAY_BE_NULL;
4654+
}
46574655
if (orig_op1_type != IS_UNKNOWN) {
46584656
if (orig_op1_type & IS_TRACE_REFERENCE) {
46594657
if (!zend_jit_fetch_reference(&dasm_state, opline, orig_op1_type, &op1_info, &op1_addr,
Lines changed: 22 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,22 @@
1+
--TEST--
2+
Register Alloction 012: Missed type store
3+
--INI--
4+
opcache.enable=1
5+
opcache.enable_cli=1
6+
opcache.file_update_protection=0
7+
opcache.jit_buffer_size=1M
8+
--FILE--
9+
<?php
10+
function foo($a) {
11+
$b = $a;
12+
$b =! $a = $a + $b & $b & $bb = $a = $a + $b & $a;
13+
$a = $a + $b & $b & $b = $a;
14+
}
15+
16+
for ($i = 0; $i < 3; $i++) {
17+
@foo(39087589046889428661);
18+
}
19+
?>
20+
DONE
21+
--EXPECTF--
22+
DONE

0 commit comments

Comments
 (0)