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

Click on Brave Rewards is causing the brave crash in the Guest window #14995

Closed
GeetaSarvadnya opened this issue Mar 29, 2021 · 4 comments
Closed
Assignees
Labels
closed/duplicate Issue has already been reported

Comments

@GeetaSarvadnya
Copy link

GeetaSarvadnya commented Mar 29, 2021

Description

Click on Brave Rewards is causing the brave crash in the Guest window

Steps to Reproduce

  1. Open 1.24.x or 1.22.x
  2. Open a Guest window
  3. Click on settings (from the hamburger menu in the Guest window)
  4. Click on Brave Rewards header
  5. Brave crashes

Actual result:

Brave Crash
Brave Rewards header_Brave crash

1.22.x

Uploaded Crash Report ID: 9a280000-0c88-1106-0000-000000000000
Upload Time: Monday, March 29, 2021 at 3:09:43 PM

1.24.x

Uploaded Crash Report ID: a3280000-0c88-1106-0000-000000000000
Upload Time: Monday, March 29, 2021 at 3:17:10 PM

Expected result:

No crash

Reproduces how often:

Easy

Brave version (brave://version info)

Brave 1.24.29 Chromium: 90.0.4430.41 (Official Build) nightly (64-bit)
Revision e9c92b1eaca8487e212f3f6bc081fdb6d4863759-refs/branch-heads/4430@{#723}
OS Windows 10 OS Version 2004 (Build 19041.867)
Brave 1.22.70 Chromium: 89.0.4389.105 (Official Build) (64-bit)
Revision 14f44e21a9d539cd49c72468a29bfca4fa43f710-refs/branch-heads/4389_90@{#7}
OS Windows 10 OS Version 2004 (Build 19041.867)

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? NA
  • Does the issue resolve itself when disabling Brave Rewards? NA
  • Is the issue reproducible on the latest version of Chrome? NA

Miscellaneous Information:

cc: @brave/legacy_qa @iccub @emerick @rebron @zenparsing

@emerick
Copy link
Contributor

emerick commented Mar 31, 2021

Haven't been able to repro yet. Backtrace indicates that it's crashing with EXCEPTION_ACCESS_VIOLATION_READ when calling the following in BraveRewardsPageUI::BraveRewardsPageUI:

handler->Init();

https://brave.sp.backtrace.io/p/brave/debug?time=all&filters=(_rxid%3Da3280000-0c88-1106-0000-000000000000)&debug=(c49e1b,0,0)

@emerick emerick self-assigned this Mar 31, 2021
@emerick
Copy link
Contributor

emerick commented Mar 31, 2021

OK, so we're dying in RewardsDOMHandler::Init because brave_rewards::RewardsServiceFactory::GetForProfile(profile) returns a nullptr for a guest page. But that's supposed to happen because Rewards isn't supported in guest pages.

The deeper question here is why the settings page is opening within a guest page. I thought it would redirect to a regular profile and open there. That's how this works in Chrome.

@emerick
Copy link
Contributor

emerick commented Mar 31, 2021

After speaking with @darkdh and @rebron, this is not a Rewards bug as such so closing this in favor of #15053.

@LaurenWags
Copy link
Member

Updated issue labels/milestone per discussion with @emerick 👍🏻

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
closed/duplicate Issue has already been reported
Projects
None yet
Development

No branches or pull requests

4 participants