diff options
author | Andriy Sultanov <53952748+last-genius@users.noreply.github.com> | 2024-09-05 04:25:49 +0100 |
---|---|---|
committer | GitHub <noreply@github.com> | 2024-09-05 03:25:49 +0000 |
commit | aadc212b95cdef2a25475f025aa7fab659dde646 (patch) | |
tree | e3123db81d75a67f75e3e84934fd069cdeb7ad01 /components/script/dom/bindings/proxyhandler.rs | |
parent | 642c25d9a73b2c3fbe6f98a32bae5c7bdb6c448f (diff) | |
download | servo-aadc212b95cdef2a25475f025aa7fab659dde646.tar.gz servo-aadc212b95cdef2a25475f025aa7fab659dde646.zip |
jsstring_to_str should accept a NonNull argument for the JS string (#33306)
Instead of asserting the raw pointer is not null, force callers to
produce a NonNull pointer.
Signed-off-by: Andriy Sultanov <sultanovandriy@gmail.com>
Diffstat (limited to 'components/script/dom/bindings/proxyhandler.rs')
-rw-r--r-- | components/script/dom/bindings/proxyhandler.rs | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/components/script/dom/bindings/proxyhandler.rs b/components/script/dom/bindings/proxyhandler.rs index 2baa5374227..315cff7bafb 100644 --- a/components/script/dom/bindings/proxyhandler.rs +++ b/components/script/dom/bindings/proxyhandler.rs @@ -259,7 +259,7 @@ unsafe fn id_to_source(cx: SafeJSContext, id: RawHandleId) -> Option<DOMString> jsstr.set(jsapi::JS_ValueToSource(*cx, value.handle().into())); jsstr.get() }) - .filter(|jsstr| !jsstr.is_null()) + .and_then(|jsstr| ptr::NonNull::new(jsstr)) .map(|jsstr| jsstring_to_str(*cx, jsstr)) } |