Skip to content

Fix EXC_BAD_ACCESS within RCTInspectorNetworkReporter#50952

Closed
sanjay-io wants to merge 1 commit intofacebook:mainfrom
sanjay-io:export-D73720844
Closed

Fix EXC_BAD_ACCESS within RCTInspectorNetworkReporter#50952
sanjay-io wants to merge 1 commit intofacebook:mainfrom
sanjay-io:export-D73720844

Conversation

@sanjay-io
Copy link
Copy Markdown

@sanjay-io sanjay-io commented Apr 28, 2025

Changelog: [Internal]

Summary:
This attempts to fix a crash introduced a few weeks ago:

https://fburl.com/logview/8s19uu9b

Sapienz caught the crash over here:

D71470038

You can see on line 664, we do a check for task.requestId, but not on the line that we added a safety around.

Differential Revision: D73720844
@facebook-github-bot facebook-github-bot added CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. p: Facebook Partner: Facebook Partner labels Apr 28, 2025
@facebook-github-bot
Copy link
Copy Markdown
Contributor

This pull request was exported from Phabricator. Differential Revision: D73720844

@react-native-bot
Copy link
Copy Markdown
Collaborator

Fails
🚫

❔ This pull request needs a description.

Warnings
⚠️ 📋 Missing Test Plan - Can you add a Test Plan? To do so, add a "## Test Plan" section to your PR description. A Test Plan lets us know how these changes were tested.

Generated by 🚫 dangerJS against a733429

@facebook-github-bot facebook-github-bot added the Merged This PR has been merged. label Apr 28, 2025
@facebook-github-bot
Copy link
Copy Markdown
Contributor

This pull request has been merged in 7a919d6.

@react-native-bot
Copy link
Copy Markdown
Collaborator

This pull request was successfully merged by Sanjay Thakur in 7a919d6

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

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. fb-exported Merged This PR has been merged. p: Facebook Partner: Facebook Partner

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants