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

Crash when timeout finishes #998

Closed
mfikes opened this issue Jul 25, 2019 · 2 comments · Fixed by #999
Closed

Crash when timeout finishes #998

mfikes opened this issue Jul 25, 2019 · 2 comments · Fixed by #999

Comments

@mfikes
Copy link
Member

mfikes commented Jul 25, 2019

It appears that the creation of the number on timeout is causing a crash:

  thread #115, stop reason = signal SIGSTOP
    frame #0: 0x00007fff4d7e9693 JavaScriptCore`WTFCrashWithInfo(int, char const*, char const*, int) + 19
    frame #1: 0x00007fff4d64ba57 JavaScriptCore`JSC::JSLockHolder::~JSLockHolder() + 311
    frame #2: 0x00007fff4d75cb56 JavaScriptCore`JSValueMakeNumber + 150
    frame #3: 0x0000000100d47354 planck`do_run_timeout(data=0x00007fa4b8034020) at functions.c:1264:15
    frame #4: 0x0000000100d555b3 planck`timer_thread(data=0x00007fa4b80e0880) at timers.c:35:5
    frame #5: 0x00007fff764ca2eb libsystem_pthread.dylib`_pthread_body + 126
    frame #6: 0x00007fff764cd249 libsystem_pthread.dylib`_pthread_start + 66
    frame #7: 0x00007fff764c940d libsystem_pthread.dylib`thread_start + 13
@mfikes
Copy link
Member Author

mfikes commented Jul 25, 2019

This reproduces with leonoel/missionary@02325aa and when running the following in that repo:

clojure -Acljs-test -x planck

In that case you will see that the underlying planck process dies early.

mfikes added a commit that referenced this issue Jul 25, 2019
@mfikes
Copy link
Member Author

mfikes commented Jul 25, 2019

Note, to diagnose this on macOS involved

ulimit -c unlimited

Then causing the repro, which produces a core in /cores.

Then you can use lldb ...path/to/planck -c /cores/<corename and then bt all to see all of the stacktraces.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant