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

Instant crash when opening brave://adblock in a private window #24086

Closed
fmarier opened this issue Jul 16, 2022 · 5 comments · Fixed by brave/brave-core#14321
Closed

Instant crash when opening brave://adblock in a private window #24086

fmarier opened this issue Jul 16, 2022 · 5 comments · Fixed by brave/brave-core#14321

Comments

@fmarier
Copy link
Member

fmarier commented Jul 16, 2022

Steps to Reproduce

  1. Start Brave.
  2. Open a new private window.
  3. Close all other windows.
  4. Open brave://adblock in the private window.

Please also check other window types as per brave/brave-core#14321 (comment).

Actual result:

Crashes the browser. Crash ID 7a570200-2312-8508-0000-000000000000.

Expected result:

Opens a normal window and then opens the settings page there.

Reproduces how often:

100% reproducible

Brave version (brave://version info)

Brave	1.43.23 Chromium: 103.0.5060.114 (Official Build) nightly (64-bit) 
Revision	a1c2360c5b02a6d4d6ab33796ad8a268a6128226-refs/branch-heads/5060@{#1124}
OS	Linux
@rebron rebron added the priority/P2 A bad problem. We might uplift this to the next planned release. label Jul 19, 2022
@GeetaSarvadnya
Copy link

GeetaSarvadnya commented Jul 19, 2022

Selecting the Manage custom filters context menu in the Private/TOR window is causing the crash. The issue is happening in RC (1.41.x) as well as in beta (1.42.x)

crash in cosmetic filtering

@rebron
Copy link
Collaborator

rebron commented Jul 19, 2022

cc: @nullhook @iefremov

@nullhook
Copy link

the crash looks related to identity manager? trace is below

[70992:259:0725/100239.376986:FATAL:people_handler.cc(891)] Check failed: identity_manager. 
0   libbase.dylib                       0x0000000104ed5a4c base::debug::CollectStackTrace(void**, unsigned long) + 12
1   libbase.dylib                       0x0000000104dcb3f0 base::debug::StackTrace::StackTrace() + 24
2   libbase.dylib                       0x0000000104deeb58 logging::LogMessage::~LogMessage() + 188
3   libbase.dylib                       0x0000000104def7a4 logging::LogMessage::~LogMessage() + 12
4   libbase.dylib                       0x0000000104db8754 logging::CheckError::~CheckError() + 36
5   libchrome_dll.dylib                 0x0000000110a3d244 settings::PeopleHandler::GetSyncStatusDictionary() const + 220
6   libchrome_dll.dylib                 0x0000000110a3baec settings::PeopleHandler::HandleGetSyncStatus(base::Value::List const&) + 176
7   libcontent.dylib                    0x00000001182825c8 content::WebUIImpl::ProcessWebUIMessage(GURL const&, std::Cr::basic_string<char, std::Cr::char_traits<char>, std::Cr::allocator<char>> const&, base::Value::List) + 156
8   libcontent.dylib                    0x0000000118281468 content::WebUIImpl::Send(std::Cr::basic_string<char, std::Cr::char_traits<char>, std::Cr::allocator<char>> const&, base::Value::List) + 268
9   libcontent.dylib                    0x00000001178e3c90 content::mojom::WebUIHostStubDispatch::Accept(content::mojom::WebUIHost*, mojo::Message*) + 360
10  libbindings.dylib                   0x00000001046934ac mojo::InterfaceEndpointClient::HandleValidatedMessage(mojo::Message*) + 1132
11  libbindings.dylib                   0x00000001046995f8 mojo::MessageDispatcher::Accept(mojo::Message*) + 312
12  libbindings.dylib                   0x0000000104694ccc mojo::InterfaceEndpointClient::HandleIncomingMessage(mojo::Message*) + 144
13  libipc.dylib                        0x00000001061f6504 IPC::(anonymous namespace)::ChannelAssociatedGroupController::AcceptOnEndpointThread(mojo::Message) + 620
14  libipc.dylib                        0x00000001061f3b20 base::internal::Invoker<base::internal::BindState<void (IPC::(anonymous namespace)::ChannelAssociatedGroupController::*)(mojo::Message), scoped_refptr<IPC::(anonymous namespace)::ChannelAssociatedGroupController>, mojo::Message>, void ()>::RunOnce(base::internal::BindStateBase*) + 76

@kjozwiak
Copy link
Member

The above requires 1.42.81 or higher for 1.42.x verification 👍

@LaurenWags
Copy link
Member

LaurenWags commented Jul 28, 2022

Verified with

Brave | 1.42.81 Chromium: 104.0.5112.69 (Official Build) (x86_64)
-- | --
Revision | 7ce2902023c722af8564068e6b26e934b83fd774-refs/branch-heads/5112@{#1213}
OS | macOS Version 12.5 (Build 21G72)

Reproduced the issue using STR and 1.41.100 Chromium: 103.0.5060.134.

Per STR in #24086 (comment), confirmed no crash under additional window types as was checked prior to uplift via brave/brave-core#14321 (comment):

  • ensured that Brave doesn't crash when loading brave://adblock via Private windows
  • ensured that Brave doesn't crash when loading brave://adblock via Tor windows
  • ensured that Brave doesn't crash when loading brave://adblock via Guest windows
  • ensured that Brave doesn't crash when loading brave://adblock via Normal windows
Example (Private) Example (Tor) Example (Guest) Example (Normal)
Private Tor Guest Normal

Verification PASSED on

Brave | 1.42.81 Chromium: 104.0.5112.69 (Official Build) (64-bit)
-- | --
Revision | 7ce2902023c722af8564068e6b26e934b83fd774-refs/branch-heads/5112@{#1213}
OS | Windows 10 Version 21H2 (Build 19044.1826)

Per STR in #24086 (comment), confirmed no crash under additional window types as was checked prior to uplift via brave/brave-core#14321 (comment):

  • ensured that Brave doesn't crash when loading brave://adblock via Private windows
  • ensured that Brave doesn't crash when loading brave://adblock via Tor windows
  • ensured that Brave doesn't crash when loading brave://adblock via Guest windows
  • ensured that Brave doesn't crash when loading brave://adblock via Normal windows
Example (Private) Example (Tor) Example (Guest) Example (Normal)
image image image image

Verification passed on

Brave 1.42.81 Chromium: 104.0.5112.69 (Official Build) (64-bit)
Revision 7ce2902023c722af8564068e6b26e934b83fd774-refs/branch-heads/5112@{#1213}
OS Ubuntu 18.04 LTS

Per STR in #24086 (comment), confirmed no crash under additional window types as was checked prior to uplift via brave/brave-core#14321 (comment):

  • ensured that Brave doesn't crash when loading brave://adblock via Private windows
  • ensured that Brave doesn't crash when loading brave://adblock via Tor windows
  • ensured that Brave doesn't crash when loading brave://adblock via Guest windows
  • ensured that Brave doesn't crash when loading brave://adblock via Normal windows

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

Successfully merging a pull request may close this issue.

9 participants