Skip to content

Commit c068963

Browse files
committed
Review comments
1 parent 6efa5a5 commit c068963

File tree

1 file changed

+4
-2
lines changed

1 file changed

+4
-2
lines changed

Zend/zend_execute.c

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1384,7 +1384,7 @@ static zend_never_inline zend_long zend_check_string_offset(zval *dim, int type
13841384
return offset;
13851385
}
13861386
zend_illegal_string_offset(dim);
1387-
break;
1387+
return 0;
13881388
}
13891389
case IS_UNDEF:
13901390
ZVAL_UNDEFINED_OP2();
@@ -2384,7 +2384,8 @@ static zend_always_inline void zend_fetch_dimension_address_read(zval *result, z
23842384
return;
23852385
}
23862386
zend_illegal_string_offset(dim);
2387-
break;
2387+
ZVAL_NULL(result);
2388+
return;
23882389
}
23892390
case IS_UNDEF:
23902391
ZVAL_UNDEFINED_OP2();
@@ -2401,6 +2402,7 @@ static zend_always_inline void zend_fetch_dimension_address_read(zval *result, z
24012402
goto try_string_offset;
24022403
default:
24032404
zend_illegal_string_offset(dim);
2405+
ZVAL_NULL(result);
24042406
return;
24052407
}
24062408

0 commit comments

Comments
 (0)