Skip to content

Swift: WebView JS-native bridge sources #11027

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

Conversation

atorralba
Copy link
Contributor

Adds taint sources for JS-native bridges in WebViews. These are objects where JavaScript running in the WebView can execute native code, which can be exploited by malicious websites, or external attackers through XSS vulnerabilities, if it performs sensitive actions.

@atorralba atorralba requested a review from a team as a code owner October 28, 2022 09:00
@github-actions github-actions bot added the Swift label Oct 28, 2022
This is what contains externally-provided data in Webview JS-native bridges
@atorralba atorralba force-pushed the atorralba/swift/webview-js-native-bridge-sources branch from 5557744 to 5ef8f9c Compare October 28, 2022 13:03
Better model the JSExport protocol logic
@atorralba atorralba force-pushed the atorralba/swift/webview-js-native-bridge-sources branch from 5ef8f9c to baf7986 Compare October 28, 2022 13:56
Copy link
Contributor

@geoffw0 geoffw0 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is complicated stuff, but they look like fantastic taint sources. A few comments / questions...

Model the source as an access to the tainted field, instead of the field itself (which didn't work)
Copy link
Contributor

@geoffw0 geoffw0 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm happy with this and I'm really glad to have more remote flow sources!

@atorralba atorralba merged commit 759ffc4 into github:main Nov 2, 2022
@atorralba atorralba deleted the atorralba/swift/webview-js-native-bridge-sources branch November 2, 2022 08:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants