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

"Framework!ProfileDestroyer::~ProfileDestroyer()" crash when closing Tor windows #8067

Closed
kjozwiak opened this issue Feb 5, 2020 · 4 comments

Comments

@kjozwiak
Copy link
Member

kjozwiak commented Feb 5, 2020

Description

Originally found/reported by @yrliou. When closing opened Tor windows on CR 80.0.3987.78, the browser will crash via the following:

 Brave Browser Framework!ProfileDestroyer::~ProfileDestroyer() [profile_destroyer.cc : 168 + 0x0]

Seems like this is happening on CR 80.0.3987.66 and CR 80.0.3987.78 but NOT CR 80.0.3987.85

Crashes:

@brave/legacy_qa lets wait till we get a build with 80.0.3987.85 and ensure Tor is working as expected.

Steps to Reproduce

  1. install Brave that's running CR 80.0.3987.66 or CR 80.0.3987.78
  2. open a Tor window
  3. close the Tor window

Actual result:

When closing Tor windows, Brave will crash as described above.

Expected result:

Brave shouldn't be crashing when closing Tor windows.

Reproduces how often:

100% reproducible using the above STR.

Brave version (brave://version info)

Brave 1.3.110 Chromium: 80.0.3987.78 (Official Build) (64-bit)
Revision cb5aa81cb0a3a2c674b93bc7ffb34a86cd3b4802-refs/branch-heads/3987@{#752}
OS macOS Version 10.15.3 (Build 19D76)

Version/Channel Information:

  • Can you reproduce this issue with the current release? N/A (doesn't have CR80)
  • Can you reproduce this issue with the beta channel? Yes
  • Can you reproduce this issue with the dev channel? Yes
  • 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 @rebron @yrliou @brave/legacy_qa @mkarolin

@kjozwiak
Copy link
Member Author

kjozwiak commented Feb 5, 2020

As per the above, @yrliou mentioned that this seems to be working under CR 80.0.3987.85. Created this issue so QA makes sure that Tor is working as expected once we get builds running 80.0.3987.85.

@yrliou
Copy link
Member

yrliou commented Feb 5, 2020

Tried with v1.3.111 (CR 80.0.3987.85) and didn't see any crashes when closing Tor window.

@kjozwiak kjozwiak added the QA/Yes label Feb 5, 2020
@kjozwiak
Copy link
Member Author

kjozwiak commented Feb 5, 2020

Closing as per above 👍 Looks like it was addressed with 80.0.3987.85 as per #8067 (comment). QA will go through the Tor manual passes and ensure everything else is working as expected.

@kjozwiak
Copy link
Member Author

kjozwiak commented Feb 5, 2020

Verification PASSED on macOS 10.15.3 x64 using the following build:

Brave 1.3.111 Chromium: 80.0.3987.85 (Official Build) (64-bit)
Revision 583b05dfa4ffc657f0f7c5cc13f53aa17c9a5bcf-refs/branch-heads/3987@{#791}
OS macOS Version 10.15.3 (Build 19D76)

Verification passed on

Brave 1.3.111 Chromium: 80.0.3987.85 (Official Build) (64-bit)
Revision 583b05dfa4ffc657f0f7c5cc13f53aa17c9a5bcf-refs/branch-heads/3987@{#791}
OS Ubuntu 18.04 LTS

Verification passed on

Brave 1.3.111 Chromium: 80.0.3987.85 (Official Build) (64-bit)
Revision 583b05dfa4ffc657f0f7c5cc13f53aa17c9a5bcf-refs/branch-heads/3987@{#791}
OS Windows 10 OS Version 1803 (Build 17134.1006)

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

4 participants