RESOLVED FIXED 250511
JSValueGetType() is incorrect for BigInt values
https://bugs.webkit.org/show_bug.cgi?id=250511
Summary JSValueGetType() is incorrect for BigInt values
Kasper Isager Dalsgarð
Reported 2023-01-12 05:47:02 PST
JSValueGetType() currently has no path for values of type BigInt and therefore falls through to the object case, returning kJSTypeObject: https://github.com/WebKit/WebKit/blob/8196614cb0623111b43d1041469eb3247e5623bb/Source/JavaScriptCore/API/JSValueRef.cpp#L51-L79 I imagine a kJSTypeBigInt value should be introduced and a path for jsValue.isBigInt() added.
Attachments
Radar WebKit Bug Importer
Comment 1 2023-01-12 14:20:13 PST
Mark Lam
Comment 2 2023-01-12 14:25:22 PST
Hmmm, JSBigInt directly extends JSCell, and therefore, not is not a JSObject. We should fix this.
Kasper Isager Dalsgarð
Comment 3 2023-01-13 01:52:59 PST
On a related note, JSValueIsBigInt() would be great as well.
Kasper Isager Dalsgarð
Comment 4 2023-01-13 01:56:11 PST
The same goes for variants of JSValueMakeBigInt*() and JSValueToBigInt*(); those are sorely missed as well.
Mark Lam
Comment 5 2023-01-17 10:48:07 PST
Adding JSValueMakeBigInt*() and JSValueToBigInt*() is going to take a lot more time to make sure the API is correct.
Kasper Isager Dalsgarð
Comment 6 2023-01-17 10:59:35 PST
For converting BigInts to C primitives, there already seem to be private APIs available: https://github.com/WebKit/WebKit/blob/4abc58df4642909ba616e53f76f7e69b18934b18/Source/JavaScriptCore/runtime/JSCJSValue.h#L312-L313
Mark Lam
Comment 7 2023-01-17 11:19:23 PST
(In reply to Kasper Isager Dalsgarð from comment #6) > For converting BigInts to C primitives, there already seem to be private > APIs available: > > https://github.com/WebKit/WebKit/blob/ > 4abc58df4642909ba616e53f76f7e69b18934b18/Source/JavaScriptCore/runtime/ > JSCJSValue.h#L312-L313 The complication isn't with the primitives. It's with getting the API correct e.g. for interactions with other API, for correctness, for security, for performance, etc. Designing that and testing it will take time.
Kasper Isager Dalsgarð
Comment 8 2023-01-17 11:22:03 PST
Noted! It also wasn't directly relevant to the initial bug report, so I'll open separate tickets.
Kasper Isager Dalsgarð
Comment 9 2023-05-23 00:18:12 PDT
Is there any news on this?
Kasper Isager Dalsgarð
Comment 10 2023-09-03 03:36:38 PDT
Do let me know if there's anything I can do to help get this fixed. I'd be more than happy to provide a patch as well. Thanks!
Yijia Huang
Comment 11 2023-10-18 15:03:36 PDT
Kasper Isager Dalsgarð
Comment 12 2023-10-18 22:14:36 PDT
That's fantastic, thanks so much! That also deals with most of https://bugs.webkit.org/show_bug.cgi?id=250719, with the exception of conversion back to numbers.
Yijia Huang
Comment 13 2024-01-23 11:29:44 PST
*** Bug 250719 has been marked as a duplicate of this bug. ***
EWS
Comment 14 2024-05-02 13:14:50 PDT
Committed 278275@main (c97a08d9e56f): <https://commits.webkit.org/278275@main> Reviewed commits have been landed. Closing PR #19251 and removing active labels.
Yijia Huang
Comment 15 2024-05-03 12:18:20 PDT
Reopened Bugzilla. Failed Xcode build, tracking revert in https://bugs.webkit.org/show_bug.cgi?id=273705.
Yijia Huang
Comment 16 2024-05-03 13:25:55 PDT
EWS
Comment 17 2024-05-09 14:43:19 PDT
Committed 278588@main (e8e344010871): <https://commits.webkit.org/278588@main> Reviewed commits have been landed. Closing PR #28121 and removing active labels.
Note You need to log in before you can comment on or make changes to this bug.