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.

Sign up for GitHub

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

Jump to bottom

Fixed Tapjacking for Full occlusion #2443

Merged
merged 1 commit into from Aug 30, 2023
Merged

Fixed Tapjacking for Full occlusion #2443

merged 1 commit into from Aug 30, 2023

Conversation

DenBond7
Copy link
Collaborator

@DenBond7 DenBond7 commented Aug 30, 2023 •
edited by ioanmo226

This PR Fixed Tapjacking for Full occlusion

close #2441

Tests (delete all except exactly one):

  • Difficult to test (explain why). It requires a separate app with Tapjacking funtionality. Tested manually.

To be filled by reviewers

I have reviewed that this PR... (tick whichever items you personally focused on during this review):

  • addresses the issue it closes (if any)
  • code is readable and understandable
  • is accompanied with tests, or tests are not needed
  • is free of vulnerabilities

@DenBond7 DenBond7 marked this pull request as ready for review August 30, 2023 11:43
@ioanmo226 ioanmo226 merged commit c2b7471 into master Aug 30, 2023
7 checks passed
@ioanmo226 ioanmo226 deleted the issue_2441_handle_tapjacking_vulnerability branch August 30, 2023 11:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ioanmo226 ioanmo226 ioanmo226 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Fix tapjacking vulnerability
2 participants
@DenBond7 @ioanmo226