Modified Files:
js_bbs.cpp js_com.c js_console.cpp js_file.c js_global.c
js_msgbase.c js_queue.c js_socket.c js_system.c js_user.c
main.cpp sbbs.h
Log Message:
Define and use a wrapper for JS_GetInstancePrivate(): js_GetClassPrivate()
Use this in place of JS_GetPrivate() in native class methods that need the class instance's private data pointer and will do bad things if that pointer points to something other than what is expected. mcmlxxix (matt) discovered that using Object.apply(), you can invoke class methods where the 'this' instance is a different class. This would result in
"Internal Error: No Private Data." or a crash.
So now, gracefully detect this condition and report a meaningful error: "'<class-name>' instance: No Private Data or Class Mismatch"
Also, important to note: if the method uses JS_THIS_OBJECT to get the JSObject* to pass to JS_Get*Private, then it must do this *before* it calls JS_SET_RVAL.
From jsapi.h:
* NB: there is an anti-dependency between JS_CALLEE and JS_SET_RVAL: native
* methods that may inspect their callee must defer setting their return value
* until after any such possible inspection. Otherwise the return value will be
* inspected instead of the callee function object.
The js_crypt*.c files still need this treatment.
---
þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
Who's Online
Recent Visitors
System Info
Sysop:
Ruben Figueroa
Location:
Mesquite, Tx
Users:
2
Nodes:
4 (0
/
4)
Uptime:
56:17:38
Calls:
77
Files:
49
D/L today:
31 files
(1K bytes)
Messages:
62,758
Synchronet Oneliners
Denn@outwest Tue Apr 23 2024 09:39:00 Currently using Synchronet 3.17, moving up to 3.19
TheNerd@canadian Wed Apr 24 2024 17:48:20 Greetings from nerdrage.ddns.net