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

Brave crashing with "brave_shields::AdBlockBaseService::OnDATFileDataReady()" when "Continue where you left off"launching using #5309

Closed
catacavaco opened this issue Jul 20, 2019 · 6 comments

Comments

@catacavaco
Copy link

Description

Brave randomly crashes when started with the "Continue where you left off" option failing to restore the tabs previously open.
Brave then auto closes and when started up again, the tabs that should be restored from previous session are no more.
That includes pinned tabs.

Steps to Reproduce

Since this does not happen every time the computer is started up, these are the steps to reproduce when it happens

  1. Computer is shut down, boot up to Windows 10
  2. Open Brave the first time, browser crashes and closes automatically
  3. Open Brave second time, tabs are gone

Actual result:

Clean window of Brave with no tabs

Expected result:

Tabs from previous session
or
Recovery tab with option to restore previously open tabs

Reproduces how often:

Intermittent issue

Brave version (brave://version info)

Brave 0.66.100 Chromium: 75.0.3770.142 (Official Build) (64-bit)
Revision 8d93477bd18154c67a70a67019cb8032382c55dd-refs/branch-heads/3770@{#1160}
OS Windows 10 OS Version 1903 (Build 18362.239)

Other Additional Information:

  • Is the issue reproducible on the latest version of Chrome? No
    Whenever Chrome (or Chromium) crashes, it displays a recovery tab where one can reopen tabs from the previous session
@kjozwiak kjozwiak added bug crash needs-investigation A bug not 100% confirmed/fixed labels Jul 22, 2019
@kjozwiak
Copy link
Member

@catacavaco when you run into the above crash, mind going into brave://crashes and pasting the Crash Report ID into the issue so one of the devs can take a look? Example of a submitted crash report:

Screen Shot 2019-07-21 at 8 23 31 PM

Might be related to #4416 regarding sessions not restoring after certain crashes occur.

@catacavaco
Copy link
Author

Probably all the same issue

Uploaded Crash Report ID 44038cdb28436c82 (Local Context: 8a328a8b-01e0-45aa-ab72-cae597df81fd)
Crash report captured on Saturday, July 20, 2019 at 12:09:53 PM, uploaded on Monday, July 22, 2019 at 5:51:49 PM

Uploaded Crash Report ID 4b21030b8ffd74f7 (Local Context: a28c0cf3-e5ee-4665-900b-f4073cac34ac)
Crash report captured on Thursday, July 11, 2019 at 5:47:13 PM, uploaded on Monday, July 22, 2019 at 5:51:49 PM

Uploaded Crash Report ID 6f38f0ea738b3f3f (Local Context: 365bbd11-2412-4fcc-9395-eda5acb91570)
Crash report captured on Tuesday, July 9, 2019 at 7:12:17 PM, uploaded on Monday, July 22, 2019 at 5:51:49 PM

Uploaded Crash Report ID 288175601b739ac4 (Local Context: 1e8f8028-ac6b-4304-993f-6fb7688882ab)
Crash report captured on Tuesday, July 2, 2019 at 7:32:46 PM, uploaded on Monday, July 22, 2019 at 5:51:49 PM

@kjozwiak
Copy link
Member

kjozwiak commented Jul 23, 2019

Thanks for the crash reports @catacavaco Seems like it's crashing via:

brave_shields::AdBlockBaseService::OnDATFileDataReady() [ad_block_base_service.cc]

Added links to the actual crashes so it's easier for whoever investigates:

@kjozwiak kjozwiak changed the title "Continue where you left off" crash at startup Brave crashing with "brave_shields::AdBlockBaseService::OnDATFileDataReady() [ad_block_base_service.cc" when "Continue where you left off"launching using Jul 23, 2019
@kjozwiak kjozwiak changed the title Brave crashing with "brave_shields::AdBlockBaseService::OnDATFileDataReady() [ad_block_base_service.cc" when "Continue where you left off"launching using Brave crashing with "brave_shields::AdBlockBaseService::OnDATFileDataReady()" when "Continue where you left off"launching using Jul 23, 2019
@simonhong
Copy link
Member

@kjozwiak I think this crash log is very similar with #4764. And That crash seems disappeared with rust based adblock library since 0.68.

@rebron rebron added this to the 0.68.x - Beta milestone Aug 9, 2019
@rebron rebron added the QA/No label Aug 9, 2019
@rebron
Copy link
Collaborator

rebron commented Aug 9, 2019

Fixed in 0.68.

@rebron rebron closed this as completed Aug 9, 2019
@catacavaco
Copy link
Author

Downloaded beta version 0.68.121 Chromium: 76.0.3809.100 (Official Build) beta (64-bit) and indeed it is fixed, thanks for the help :)

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

No branches or pull requests

5 participants