Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Bug]: segfault in test suite #2248

Open
remicollet opened this issue Sep 7, 2023 · 2 comments
Open

[Bug]: segfault in test suite #2248

remicollet opened this issue Sep 7, 2023 · 2 comments
Labels
🐛 bug Something isn't working

Comments

@remicollet
Copy link
Contributor

Bug report

========DIFF========
--
     process
     Flushing trace of size 3 to send-queue for %s
     kill
005+ 
005- Killed%r\n*(Termsig=9)?%r
006+ Termsig=9
========DONE========
FAIL Force flush the traces mid-way through a trace [tests/ext/force_flush_traces.phpt] 

PHP version

8.1.23

Tracer or profiler version

0.91.0

Installed extensions

No response

Output of phpinfo()

No response

Upgrading from

No response

@remicollet remicollet added the 🐛 bug Something isn't working label Sep 7, 2023
@remicollet
Copy link
Contributor Author

Also failing (seems to be related to long path raise message being truncated)

TEST 277/434 [tests/ext/sandbox/fatal_errors_ignored_in_tracing_closure.phpt]
========DIFF========
001+ Error thrown in ddtrace's closure defined at /builddir/build/BUILD/php-pecl-datadog-trace-0.91.0/datadog_trace-0.91.0/tests/ext/sandbox/fatal_errors_ignored_in_tracing_closure.php:2 for array_sum(): Call to undefined function this_function_does_not_exist(
001- Error thrown in ddtrace's closure defined at %s:%d for array_sum(): Call to undefined function this_function_does_not_exist()
     int(100)
     array_sum
     NULL
--
========DONE========
FAIL Fatal errors are ignored inside a tracing closure (PHP 7+) [tests/ext/sandbox/fatal_errors_ignored_in_tracing_closure.phpt] 

@bwoebi
Copy link
Collaborator

bwoebi commented Sep 7, 2023

Just a quick note: This test tests behaviour under a sigkill (it's 9, not 11), just the matching in the diff doesn't match the behaviour observed, i.e. the system doesn't seem to output Killed for you. We'll take it into consideration in the test result match.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛 bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants