Skip to content

Fix: JSI Value::toString() should check the result type (#59146) - #59146

Closed
lavenzg wants to merge 1 commit into
react:mainfrom
lavenzg:export-D125707029
Closed

lavenzg wants to merge 1 commit into
react:mainfrom
lavenzg:export-D125707029

Conversation

@lavenzg

@lavenzg lavenzg commented Oct 16, 2026 •

Copy link
Copy Markdown
Contributor

Summary:

global.String could be overridden. We always need to check the return type before casting to jsi::String.

Changelog: [Internal]

Reviewed By: avp

Differential Revision: D125707029

@meta-cla meta-cla Bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Oct 16, 2026
@meta-codesync

meta-codesync Bot commented Oct 16, 2026

Copy link
Copy Markdown

@lavenzg has exported this pull request. If you are a Meta employee, you can view the originating Diff in D125707029.

@meta-codesync meta-codesync Bot changed the title Fix: JSI Value::toString() should check the result type Fix: JSI Value::toString() should check the result type (#59146) Oct 19, 2026
lavenzg added a commit to lavenzg/react-native that referenced this pull request Oct 19, 2026
Summary:

`global.String` could be overridden. We always need to check the return type before casting to `jsi::String`.

Changelog: [Internal]

Differential Revision: D125707029
lavenzg added a commit to lavenzg/react-native that referenced this pull request Oct 19, 2026
Summary:
Pull Request resolved: react#59146

`global.String` could be overridden. We always need to check the return type before casting to `jsi::String`.

Changelog: [Internal]

Differential Revision: D125707029
@lavenzg
lavenzg force-pushed the export-D125707029 branch 2 times, most recently from 3ca23be to 963c3ad Compare October 19, 2026 19:32
lavenzg added a commit to lavenzg/react-native that referenced this pull request Oct 19, 2026
Summary:

`global.String` could be overridden. We always need to check the return type before casting to `jsi::String`.

Changelog: [Internal]

Reviewed By: avp

Differential Revision: D125707029
Summary:
Pull Request resolved: react#59146

`global.String` could be overridden. We always need to check the return type before casting to `jsi::String`.

Changelog: [Internal]

Reviewed By: avp

Differential Revision: D125707029
meta-codesync Bot pushed a commit to facebook/hermes that referenced this pull request Oct 19, 2026
Summary:
X-link: react/react-native#59146

`global.String` could be overridden. We always need to check the return type before casting to `jsi::String`.

Changelog: [Internal]

Reviewed By: avp

Differential Revision: D125707029

fbshipit-source-id: 15d3f83ac23d8bceb24e9e13a1f67dc4b61bb3b8
meta-codesync Bot pushed a commit to facebook/hermes that referenced this pull request Oct 19, 2026
Summary:
X-link: react/react-native#59146

`global.String` could be overridden. We always need to check the return type before casting to `jsi::String`.

Changelog: [Internal]

Reviewed By: avp

Differential Revision: D125707029

fbshipit-source-id: 15d3f83ac23d8bceb24e9e13a1f67dc4b61bb3b8
meta-codesync Bot pushed a commit to facebook/hermes that referenced this pull request Oct 19, 2026
Summary:
X-link: react/react-native#59146

`global.String` could be overridden. We always need to check the return type before casting to `jsi::String`.

Changelog: [Internal]

Reviewed By: avp

Differential Revision: D125707029

fbshipit-source-id: 15d3f83ac23d8bceb24e9e13a1f67dc4b61bb3b8
@meta-codesync meta-codesync Bot closed this in 1fcf9b8 Oct 19, 2026
@react-native-bot

Copy link
Copy Markdown
Collaborator

This pull request was successfully merged by @lavenzg in 1fcf9b8

When will my fix make it into a release? | How to file a pick request?

@meta-codesync meta-codesync Bot added the Merged This PR has been merged. label Oct 19, 2026
@meta-codesync

meta-codesync Bot commented Oct 19, 2026

Copy link
Copy Markdown

This pull request has been merged in 1fcf9b8.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. Merged This PR has been merged. meta-exported p: Facebook Partner: Facebook Partner

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants