Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

#4998 Fix strict mode properties access issue. #5152

Closed
wants to merge 1 commit into from

Conversation

Romick2005
Copy link

@Romick2005 Romick2005 commented Oct 18, 2022

freezeMethods run reduceDescriptors method with reducer that is trying to access restricted properties for the strict mode.
Code was crashing on importing following line:
const value = obj[name];
where this restricted properties access was made.

This commit fix restricted access in strict mode for 'caller', 'callee', and 'arguments' properties.
TypeError: 'caller', 'callee', and 'arguments' properties may not be accessed on strict mode.
Also simplify named export.

Should fix #4998, #5005 and #5006.

…e to the 'caller', 'callee', and 'arguments' properties. TypeError: 'caller', 'callee', and 'arguments' properties may not be accessed on strict mode.
@bombillazo
Copy link

Hey @jasonsaayman , does this look good?

@jasonsaayman
Copy link
Member

Closing cause all the issues mentioned are already resolved. Please re-open and tag me if this issue persists or has not been solved.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Restricted in strict mode, js engine: hermes (React Native + Axios 1.0.0)
4 participants