diff options
author | Nikita Popov <nikita.ppv@gmail.com> | 2019-01-02 11:34:15 +0100 |
---|---|---|
committer | Nikita Popov <nikita.ppv@gmail.com> | 2019-01-02 11:34:15 +0100 |
commit | 613683580b9136001218818969e69742747cecb4 (patch) | |
tree | c5a9f28965cac03bd4b3d255523fa4ba94c5307f | |
parent | 91e44a27f68aa808e03d11b108ae8c8ea3e4c2f9 (diff) | |
parent | 73596c56e7905ce5b7c4f15d551361ee744cad0c (diff) | |
download | php-git-613683580b9136001218818969e69742747cecb4.tar.gz |
Merge branch 'PHP-7.2' into PHP-7.3
-rw-r--r-- | Zend/tests/bug75426.phpt | 15 | ||||
-rw-r--r-- | Zend/zend_compile.c | 7 |
2 files changed, 22 insertions, 0 deletions
diff --git a/Zend/tests/bug75426.phpt b/Zend/tests/bug75426.phpt new file mode 100644 index 0000000000..69e3f19239 --- /dev/null +++ b/Zend/tests/bug75426.phpt @@ -0,0 +1,15 @@ +--TEST-- +Bug #75426: "Cannot use empty array elements" reports wrong position +--FILE-- +<?php +$a = [ + 1, + 2, + 3, + , + 5, + 6, +]; +?> +--EXPECTF-- +Fatal error: Cannot use empty array elements in arrays in %s on line 5 diff --git a/Zend/zend_compile.c b/Zend/zend_compile.c index dd102e8198..a970718bfe 100644 --- a/Zend/zend_compile.c +++ b/Zend/zend_compile.c @@ -6995,6 +6995,7 @@ static inline void zend_ct_eval_greater(zval *result, zend_ast_kind kind, zval * static zend_bool zend_try_ct_eval_array(zval *result, zend_ast *ast) /* {{{ */ { zend_ast_list *list = zend_ast_get_list(ast); + zend_ast *last_elem_ast = NULL; uint32_t i; zend_bool is_constant = 1; @@ -7007,6 +7008,10 @@ static zend_bool zend_try_ct_eval_array(zval *result, zend_ast *ast) /* {{{ */ zend_ast *elem_ast = list->child[i]; if (elem_ast == NULL) { + /* Report error at line of last non-empty element */ + if (last_elem_ast) { + CG(zend_lineno) = zend_ast_get_lineno(last_elem_ast); + } zend_error(E_COMPILE_ERROR, "Cannot use empty array elements in arrays"); } @@ -7018,6 +7023,8 @@ static zend_bool zend_try_ct_eval_array(zval *result, zend_ast *ast) /* {{{ */ ) { is_constant = 0; } + + last_elem_ast = elem_ast; } if (!is_constant) { |