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

Crashes #78

Open
daurnimator opened this issue May 31, 2017 · 1 comment
Open

Crashes #78

daurnimator opened this issue May 31, 2017 · 1 comment

Comments

@daurnimator
Copy link

daurnimator commented May 31, 2017

#
# Fatal error in , line 0
# unreachable code
#

==== C stack trace ===============================

    node(v8::base::debug::StackTrace::StackTrace()+0x16) [0x10d4146]
    node(V8_Fatal+0xe8) [0x10d2388]
    node(v8::internal::ToBooleanICStub::Types::UpdateStatus(v8::internal::Isolate*, v8::internal::Handle<v8::internal::Object>)+0x3c) [0x8905ec]
    node(v8::internal::ToBooleanICStub::UpdateStatus(v8::internal::Handle<v8::internal::Object>)+0x3f) [0x8a22ff]
    node(v8::internal::ToBooleanIC::ToBoolean(v8::internal::Handle<v8::internal::Object>)+0x3d) [0xbe4d7d]
    node(v8::internal::Runtime_ToBooleanIC_Miss(int, v8::internal::Object**, v8::internal::Isolate*)+0xb3) [0xbe50e3]
    [0x1b3f508843a7]
Illegal instruction (core dumped)

From coredumpctl:

       Message: Process 10612 (node) of user 1000 dumped core.
                
                Stack trace of thread 10612:
                #0  0x00000000010d59f9 _ZN2v84base2OS5AbortEv (node)
                #1  0x00000000010d23aa V8_Fatal (node)
                #2  0x00000000008905ec _ZN2v88internal15ToBooleanICStub5Types12Upd
                #3  0x00000000008a22ff _ZN2v88internal15ToBooleanICStub12UpdateSta
                #4  0x0000000000be4d7d _ZN2v88internal11ToBooleanIC9ToBooleanENS0_
                #5  0x0000000000be50e3 _ZN2v88internal24Runtime_ToBooleanIC_MissEi
                #6  0x00001b3f508843a7 n/a (n/a)

Running weak version 1.0.1.

$ node -v
v7.10.0

I'm trying to narrow it down, but it seems to go away if I make the sample smaller than 2 files and ~200 lines (no matter which lines I remove)

@danfuzz
Copy link

danfuzz commented Mar 28, 2019

Note that as of a year or so ago, there is https://github.com/node-ffi-napi/weak-napi, which is API-compatible with (Nate's) weak and claims to solve the crashiness problem (and also uses N-API for better binary compatibility).

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

No branches or pull requests

2 participants