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

Crash in ProxyConfigServiceTor #16070

Closed
iefremov opened this issue May 25, 2021 · 2 comments
Closed

Crash in ProxyConfigServiceTor #16070

iefremov opened this issue May 25, 2021 · 2 comments
Labels
closed/duplicate Issue has already been reported crash feature/tor OS/Desktop priority/P3 The next thing for us to work on. It'll ride the trains.
Projects

Comments

@iefremov
Copy link
Contributor

Follow-up to #12798 that still reproduces

https://brave.sp.backtrace.io/p/brave/explore?time=month&filters=(ver%3E%3D%2290.1.23%22%2C(callstack%2Ccontains%2Cnet%3A%3AProxyConfigServiceTor%3A%3ASetProxyAuthorization))&aggregations=((ver%2Cmax)%2C(callstack%2Chead))&

[ 00 ] <name omitted>
[ 01 ] net::ProxyConfigServiceTor::SetProxyAuthorization(net::ProxyConfigWithAnnotation const&, GURL const&, net::ProxyResolutionService*, net::ProxyInfo*)
[ 02 ] net::ConfiguredProxyResolutionService::TryToCompleteSynchronously(GURL const&, net::ProxyInfo*)
[ 03 ] net::ConfiguredProxyResolutionService::ResolveProxy(GURL const&, std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char> > const&, net::NetworkIsolationKey const&, net::ProxyInfo*, base::OnceCallback<void (int)>, std::__1::unique_ptr<net::ProxyResolutionRequest, std::__1::default_delete<net::ProxyResolutionRequest> >*, net::NetLogWithSource const&)
[ 04 ] <name omitted>
[ 05 ] network::NetworkContext::LookUpProxyForURL(GURL const&, net::NetworkIsolationKey const&, mojo::PendingRemote<network::mojom::ProxyLookupClient>)
@iefremov iefremov added crash feature/tor priority/P3 The next thing for us to work on. It'll ride the trains. OS/Desktop labels May 25, 2021
@iefremov iefremov added this to To do in Crashes via automation May 25, 2021
@iefremov
Copy link
Contributor Author

cc @darkdh

@iefremov
Copy link
Contributor Author

fixed in #37884

Crashes automation moved this from To do to Done May 14, 2024
@iefremov iefremov added the closed/duplicate Issue has already been reported label May 14, 2024
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 crash feature/tor OS/Desktop priority/P3 The next thing for us to work on. It'll ride the trains.
Projects
Development

No branches or pull requests

1 participant