Rantanen / node-mumble

Mumble client in Node.js
MIT License
155 stars 48 forks source link

Fatal error in api.cc #95

Closed Stateford closed 7 years ago

Stateford commented 7 years ago
#
# Fatal error in api.cc, line 1051
# Check failed: !value_obj->IsJSReceiver() || value_obj->IsTemplateInfo().
#

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

        v8::Testing::DeoptimizeAll [0x00007FF7DB5ED146+1835798]
        v8::Template::Set [0x00007FF7DB3FB733+387]
        NodeJitterBuffer::Init [0x00007FFD68DC2282+258] (d:\github\simp-bot-test\node_modules\jitterbuffer\src\node-jitt
erbuffer.cc:142)
        NodeInit [0x00007FFD68DC2529+9] (d:\github\simp-bot-test\node_modules\jitterbuffer\src\node-jitterbuffer.cc:167)
        node_module_register [0x00007FF7DAE66C17+2439]
        v8::Isolate::NumberOfHeapSpaces [0x00007FF7DB3F6A0F+3076927]
        v8::Isolate::NumberOfHeapSpaces [0x00007FF7DB39647E+2682286]
        v8::Isolate::NumberOfHeapSpaces [0x00007FF7DB395373+2677923]
        v8::Isolate::NumberOfHeapSpaces [0x00007FF7DB39522E+2677598]
        (No symbol) [0x00000086E4A063AB]

I'm not sure if this is something that broke in newer versions of node / npm installs, but I've been getting this error constantly.

Rantanen commented 7 years ago

I'll take a look at this in the evening.

Stateford commented 7 years ago

Thank you. This occurred on 7.0.0.

I want to avoid rolling back the node version on my server is possible.

Rantanen commented 7 years ago

I know node-jitterbuffer had some deprecation warning a while back, but when I tried debugging that I had trouble getting the thing to compile in my environment. No idea what caused that - then I forgot about it.

The stack fortunately has the line numbers for node-jitterbuffer so I'll take a look at how to fix that. I'm guessing that's the reason this is happening now.

Rantanen commented 7 years ago

Found the issue in node-jitterbuffer. Fixed it there (and in node-celt).

Stateford commented 7 years ago

working now, thanks you're awesome.

Rantanen commented 7 years ago

I might disagree with that given Node started warning about this in 6.0.0 and it was reported in August by notthetup in #92 but I never got around fixing it until it broke.

But I'm glad someone thinks that way. Thanks! :)