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

nightly crashing due to "brave_stats::BraveStatsUpdater::BuildStatsEndpoint" #13858

Closed
kjozwiak opened this issue Jan 31, 2021 · 12 comments · Fixed by brave/brave-core#7753 or brave/brave-core#7789
Assignees
Labels

Comments

@kjozwiak
Copy link
Member

kjozwiak commented Jan 31, 2021

Description

Nightly is currently crashing every few seconds after startup or after a few minutes. I have a brand new profile that crashes every ~5min but I also have a profile that crashes every ~1-2 seconds after startup making it almost impossible to use.

We're also getting several reports from the community as per the following:

@bsclifton also ran into the above issue on his machine and reported the crash via #crashes in Slack.

Here's a quick preview of the Stack Full stack can be downloaded via the following:

[ 00 ] brave_stats::BraveStatsUpdater::BuildStatsEndpoint(std::__1::basic_string<char,std::__1::char_traits<char>,std::__1::allocator<char> > const &)
[ 01 ] std::__1::basic_string<char,std::__1::char_traits<char>,std::__1::allocator<char> >::basic_string<nullptr_t>
[ 02 ] brave_stats::BraveStatsUpdater::SendServerPing()
[ 03 ] std::__1::__tree<std::__1::__value_type<Profile::OTRProfileID,std::__1::unique_ptr<Profile,std::__1::default_delete<Profile> > >,std::__1::__map_value_compare<Profile::OTRProfileID,std::__1::__value_type<Profile::OTRProfileID,std::__1::unique_ptr<Profile,std::__1::default_delete<Profile> > >,std::__1::less<Profile::OTRPro

Crash ID's from @kjozwiak:

  • 4d2a0000-9cc8-c705-0000-000000000000
  • 4c2a0000-9cc8-c705-0000-000000000000
  • 4a2a0000-9cc8-c705-0000-000000000000
  • 102a0000-9cc8-c705-0000-000000000000

Crash ID's from @bsclifton:

  • a01f0000-9cc8-c705-0000-000000000000
  • a51f0000-9cc8-c705-0000-000000000000
  • a61f0000-9cc8-c705-0000-000000000000
  • a71f0000-9cc8-c705-0000-000000000000
  • a81f0000-9cc8-c705-0000-000000000000
  • a91f0000-9cc8-c705-0000-000000000000

Steps to Reproduce

  1. launch 1.21.35 CR: 88.0.4324.96
  2. login into a few websites (GH/Gmail in my case) and wait ~10min or so
  3. eventually Nightly will crash

I have a profile saved that basically crashes after startup as well so it makes it impossible to use Brave on Nightly.

Actual result:

Nightly crashing pretty often which makes it impossible to use

Expected result:

Nightly shouldn't be crashing.

Reproduces how often:

So far I've managed to reproduce the crash on three different clean profiles after ~20min of usage.

Brave version (brave://version info)

Brave | 1.21.35 Chromium: 88.0.4324.96 (Official Build) nightly (64-bit)
-- | --
Revision | 68dba2d8a0b149a1d3afac56fa74648032bcf46b-refs/branch-heads/4324@{#1784}
OS | Windows 10 OS Version 2009 (Build 19042.746)

Version/Channel Information:

  • Can you reproduce this issue with the current release? I'm not 100% sure if this occurring on Release
  • Can you reproduce this issue with the beta channel? I'm not 100% sure if this occurring on BETA
  • Can you reproduce this issue with the nightly channel? Yes

Other Additional Information:

  • Does the issue resolve itself when disabling Brave Shields? N/A
  • Does the issue resolve itself when disabling Brave Rewards? N/A
  • Is the issue reproducible on the latest version of Chrome? N/A

Miscellaneous Information:

CCing @bsclifton @rebron @bbondy @aekeus @brave/legacy_qa @keur

@kjozwiak kjozwiak added bug crash priority/P1 A very extremely bad problem. We might push a hotfix for it. OS/Desktop labels Jan 31, 2021
@kjozwiak
Copy link
Member Author

Workaround for now is to launch Nightly via the terminal/command line using the following:

brave.exe --brave-stats-updater-server="https://laptop-updates.brave.com"

@thinginyourmind
Copy link

There is no issue on release build (Version 1.19.88 Chromium: 88.0.4324.96 (Official Build) (64-bit)), but nightly indeed crashes after 1-2 seconds, at least when using on windows 10. The workaround shown above works, indeed.

@gruber123
Copy link

I'm again updated to Nightly v1.21.35 from Nightly v1.21.31 and browser isn't crushing (I haven't used workaround through terminal yet). My system windows 7

@Bushido1
Copy link

Workaround for now is to launch Nightly via the terminal/command line using the following:

brave.exe --brave-stats-updater-server="https://laptop-updates.brave.com"

How to launch Nightly via the terminal/command line?
Inserting "C:\Program Files\BraveSoftware\Brave-Browser-Nightly\Application\brave.exe -brave-stats-updater-server="https://laptop-updates.brave.com"" to the command line did not work as attached.

image

@kjozwiak kjozwiak added this to the 1.21.x - Nightly milestone Jan 31, 2021
@kjozwiak
Copy link
Member Author

kjozwiak commented Jan 31, 2021

@Bushido1 I would do the following:

  • cd "C:\Program Files\BraveSoftware\Brave-Browser-Nightly\Application\"
  • launch brave using brave.exe --brave-stats-updater-server="https://laptop-updates.brave.com"

This should also work:

"C:\Program Files\BraveSoftware\Brave-Browser-Nightly\Application\brave.exe" --brave-stats-updater-server="https://laptop-updates.brave.com"

@141
Copy link

141 commented Jan 31, 2021

Tabs are crashing with a Status_Access_Violation error code on Version 1.19.88 Chromium: 88.0.4324.96 (Official Build) (64-bit).
Basically the say way, some tabs open, and eventually the tab you are on will crash.

@Coording
Copy link

@kjozwiak

"C:\Program Files\BraveSoftware\Brave-Browser-Nightly\Application\brave.exe" --brave-stats-updater-server="https://laptop-updates.brave.com"

This worked for me, thanks

@fruitofmight
Copy link

Workaround for now is to launch Nightly via the terminal/command line using the following:

brave.exe --brave-stats-updater-server="https://laptop-updates.brave.com"

To get around this without using the command line or cmd, you can make a shortcut of nightly on desktop or anywhere you like and then open up the properties.

Change the target to the given workaround by @kjozwiak, it is a quite good and simple workaround for the case.

@kjozwiak
Copy link
Member Author

We also released https://github.com/brave/brave-browser/releases/tag/v1.21.36 that had a quick fix as per brave/brave-core#7753. That should have resolved the issue 👍 We're also working on a longer term solution as per brave/brave-core#7754. We'll adjust our process so this doesn't happen.

bsclifton added a commit to brave/brave-core that referenced this issue Feb 2, 2021
- check stats updater URL in GN to fail quick.
- remove check which causes crash since GN check is done.

Fixes brave/brave-browser#13858
@bsclifton bsclifton added QA/No and removed QA/Yes labels Feb 2, 2021
@bsclifton
Copy link
Member

Update to QA/No and removed test plan after code was updated to check in build rather than at runtime

bsclifton added a commit to brave/brave-core that referenced this issue Feb 3, 2021
- check stats updater URL in GN to fail quick.
- remove check which causes crash since GN check is done.

Fixes brave/brave-browser#13858
@141
Copy link

141 commented Feb 6, 2021

The latest Version 1.19.92 Chromium: 88.0.4324.152 (Official Build) (64-bit), seems to have fixed the tab crashing error for me.
Version 1.19.88 was the only errored version it seems in the release versions.

@gsxgmpkt9020
Copy link

Still has this issue in v1.48.75

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment