Skip to content

Demonstrate potential crashes due to missing SAVE_OPLINE - #9934

Closed
arnaud-lb wants to merge 3 commits into
php:masterfrom
arnaud-lb:ex-opline-crash
Closed

arnaud-lb wants to merge 3 commits into
php:masterfrom
arnaud-lb:ex-opline-crash

Conversation

@arnaud-lb

Copy link
Copy Markdown
Member

See #9933

run-tests.php with `-jN` can hang if a parallel worker dies without notice. This
can happen due to fatal errors in the worker, or if the worker is killed.

- run-tests.php (main process)
  \_ run-tests.php (worker #0) // main process hangs if this one crashes
     \_ test-001.php (test-001.phpt)
* run-tests-hanging:
  Fix run-tests.php hanging when a worker process dies without notice
Comment thread Zend/zend_vm_def.h

op1 = GET_OP1_ZVAL_PTR_UNDEF(BP_VAR_R);
op2 = GET_OP2_ZVAL_PTR_UNDEF(BP_VAR_R);
SAVE_OPLINE();

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

run-tests.php itself crashes without this SAVE_OPLINE()

@arnaud-lb arnaud-lb closed this Dec 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant