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

crash when clicking on shields while using Google Meet #22814

Closed
kjozwiak opened this issue May 12, 2022 · 4 comments · Fixed by brave/brave-core#13367
Closed

crash when clicking on shields while using Google Meet #22814

kjozwiak opened this issue May 12, 2022 · 4 comments · Fixed by brave/brave-core#13367

Comments

@kjozwiak
Copy link
Member

kjozwiak commented May 12, 2022

Description

When pressing on the Brave Shields icon while you're in a Google Meeting, the browser will crash. Reproduced this several times and originally reported by @lukaslevert and @iambrianfung.

Some Uploaded Crash Report ID from @lukaslevert @iambrianfung and myself:

  • 99f50000-8c02-3208-0000-000000000000
  • e4580500-04c8-2808-0000-000000000000
  • b0e80000-8c02-3208-0000-000000000000
  • aee80000-8c02-3208-0000-000000000000

Steps to Reproduce

Prerequisite: ensure that you're logged into Google so you can start a Hangout Meeting

  1. launch brave (used 1.40.52 Chromium: 101.0.4951.64 in this case)
  2. open Gmail in a new tab and click on New Meeting --> Start Now
  3. once the meeting is running, click on the Brave Shields icon and you'll notice Brave crashes

Additional STR/Cases can be found via brave/brave-core#13367 (comment).

Actual result:

Brave crashes when clicking in on the shields icon while in a Google Meeting

Expected result:

Brave shouldn't be crashing when interacting with shields while in a Google Meeting.

Reproduces how often:

100% reproducible when using the STR/Cases outlined above.

Brave version (brave://version info)

Brave | 1.40.52 Chromium: 101.0.4951.64 (Official Build) nightly (64-bit)
-- | --
Revision | d1daa9897e1bc1d507d6be8f2346e377e5505905-refs/branch-heads/4951@{#1208}
OS | Windows 11 Version 21H2 (Build 22000.652)

Version/Channel Information:

  • Can you reproduce this issue with the current release? Yes
  • Can you reproduce this issue with the beta channel? Yes
  • Can you reproduce this issue with the nightly channel? Yes

Other Additional Information:

  • Does the issue resolve itself when disabling Brave Shields? Causing the issue/crash (disabling shields)
  • Does the issue resolve itself when disabling Brave Rewards? N/A
  • Is the issue reproducible on the latest version of Chrome? N/A
@iefremov iefremov added the priority/P2 A bad problem. We might uplift this to the next planned release. label May 12, 2022
@iefremov
Copy link
Contributor

cc @nullhook @petemill can you take a look?

@kjozwiak
Copy link
Member Author

Above requires 1.39.109 or higher for 1.39.x verification 👍

@MadhaviSeelam
Copy link

MadhaviSeelam commented May 19, 2022

Verification PASSED

Brave | 1.39.109 Chromium: 102.0.5005.50 (Official Build) (64-bit)
-- | --
Revision | f6e2cf8f59ec714bdcff8499991d55898875f287-refs/branch-heads/5005@{#648}
OS | Windows 11 Version 21H2 (Build 22000.675)

Verified Brave didn't crash when clicked on Shields while using Google Meeting
Followed STR from ##22814 (comment)
Also Validated additional STR/Cases #brave/brave-core#13367 (comment).

2022-05-19_13h27_12.mp4

@LaurenWags
Copy link
Member

LaurenWags commented May 20, 2022

Verified with

Brave | 1.39.109 Chromium: 102.0.5005.50 (Official Build) (x86_64)
-- | --
Revision | f6e2cf8f59ec714bdcff8499991d55898875f287-refs/branch-heads/5005@{#648}
OS | macOS Version 12.3.1 (Build 21E258)

Reproduced the issue using 1.38.119 Chromium: 101.0.4951.67 (Official Build) (x86_64).

Upgraded to 1.39.109 and confirmed issue did not reproduce, there was no crash when clicking on shields while in a Google Meet.

Screen Shot 2022-05-20 at 11 27 21 AM

Verification passed on

Brave 1.39.111 Chromium: 102.0.5005.61 (Official Build) (64-bit)
Revision 0e59bcc00cc4985ce39ad31c150065f159d95ad3-refs/branch-heads/5005@{#819}
OS Ubuntu 18.04 LTS

Reproduced the issue using 1.38.119.

Upgraded to 1.39.111 and confirmed issue did not reproduce, there was no crash when clicking on shields while in a Google Meet.

avinassh pushed a commit to avinassh/brave-browser-hardening that referenced this issue May 29, 2022
 - Added Solana support for account creation, sending SOL and sending SPL tokens with Brave Wallet. ([#22348](brave/brave-browser#22348))
 - Added the ability to buy with Ramp using Brave Wallet. ([#21639](brave/brave-browser#21639))
 - Added JSONSanitizer to API helper requests for Brave Wallet. ([#21831](brave/brave-browser#21831))
 - Added Dapp UI for requesting a public key and for decrypting ciphers using Brave Wallet. ([#21177](brave/brave-browser#21177))
 - Added web3_clientVersion support for Brave Wallet. ([#19278](brave/brave-browser#19278))
 - Added the ability to allow users to search sites for RSS feeds for Brave News. ([#21768](brave/brave-browser#21768))
 - Added support for blob partitioning. ([#21746](brave/brave-browser#21746))
 - Added minimum macOS version for Sparkle update process. ([#22918](brave/brave-browser#22918))
 - [Security] Blocked "window.ethereum" completely in third party iframes. ([#22686](brave/brave-browser#22686))
 - [Security] Updated Brave Wallet panel to prominently display eTLD+1 as reported on HackerOne by renekroka. ([#21787](brave/brave-browser#21787))
 - [Security] Fixed incorrect origin being displayed in Brave Wallet when a spend approval is pending. ([#19557](brave/brave-browser#19557))
 - Implemented eth_getEncryptionPublicKey for Brave Wallet. ([#19276](brave/brave-browser#19276))
 - Implemented account discovery when restoring Brave Wallet. ([#18104](brave/brave-browser#18104))
 - Updated Omaha installer version for Windows to v1.3.36.113. ([#22060](brave/brave-browser#22060))
 - Updated default IPFS configuration values. ([#22068](brave/brave-browser#22068))
 - Updated Gas Limit validation and error messaging for unapproved transactions with Brave Wallet. ([#21714](brave/brave-browser#21714))
 - Updated Brave Wallet to automatically add swap taker asset to the visible asset list. ([#21428](brave/brave-browser#21428))
 - Updated Brave Wallet portfolio network filter for multichain support. ([#20780](brave/brave-browser#20780))
 - Reduced adblock filter memory usage by optimizing unused regex rules. ([#21970](brave/brave-browser#21970))
 - Removed known Dialog Insight user tracking parameters from URLs. ([#22082](brave/brave-browser#22082))
 - Removed ability to swap ERC721 tokens with Brave Wallet. ([#21550](brave/brave-browser#21550))
 - Fixed crash which occurred when opening Brave Shields while using Google Meet. ([#22814](brave/brave-browser#22814))
 - Fixed inability to rename Solana account in Brave Wallet after it has been created. ([#22958](brave/brave-browser#22958))
 - Fixed incorrectly computed insufficient funds errors in Brave Wallet. ([#22877](brave/brave-browser#22877))
 - Fixed ERC20 and ERC721 transfers being incorrectly displayed as ETH transfers in the Brave Wallet transactions panel. ([#22044](brave/brave-browser#22044))
 - Fixed text alignment issues under the Brave Wallet "Recent transactions" panel when using long account names. ([#21216](brave/brave-browser#21216))
 - Fixed breakage in webpack build caused by OpenSSL 3.0. ([#22305](brave/brave-browser#22305))
 - Fixed two windows being opened on launch when the browser was installed without administrator privileges on Windows. ([#22179](brave/brave-browser#22179))
 - Upgraded Chromium to 102.0.5005.61. ([#22923](brave/brave-browser#22923)) ([Changelog for 102.0.5005.61](https://chromium.googlesource.com/chromium/src/+log/101.0.4951.67..102.0.5005.61?pretty=fuller&n=1000))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment