Commit 3d71e5cc authored by Jimmy Gong's avatar Jimmy Gong Committed by Commit Bot

Fix crash caused by null callback in MdnsHostLocator

- Fixes issue where switching wifi networks while starting Mdns
  HostLocator would crash chrome.

Bug: chromium:900779
Test: end to end
Change-Id: Ide5e1bb53961dcd654aedadbffc6cf2a83c5d998
Reviewed-on: https://chromium-review.googlesource.com/c/1313331Reviewed-by: default avatarBailey Berro <baileyberro@chromium.org>
Commit-Queue: jimmy gong <jimmyxgong@chromium.org>
Cr-Commit-Position: refs/heads/master@{#604753}
parent a81383e6
......@@ -59,14 +59,14 @@ void MDnsHostLocator::FindHosts(FindHostsCallback callback) {
Reset();
}
callback_ = std::move(callback);
if (!(StartListening() && CreatePtrTransaction())) {
LOG(ERROR) << "Failed to start MDnsHostLocator";
FireCallback(false /* success */);
return;
}
callback_ = std::move(callback);
}
bool MDnsHostLocator::CreatePtrTransaction() {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment