Commit f573c086 authored by Maksim Ivanov's avatar Maksim Ivanov Committed by Chromium LUCI CQ

Fix double-callback-call crash in //chromeos/services/network_config/

Fix bug found by the "bugprone-use-after-move" clang-tidy check.

Bug: 1122844
Change-Id: I3420b6c260b9fa94847b9b63df0438a9bce0b1c7
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2556838
Commit-Queue: Steven Bennetts <stevenjb@chromium.org>
Auto-Submit: Maksim Ivanov <emaxx@chromium.org>
Reviewed-by: default avatarSteven Bennetts <stevenjb@chromium.org>
Cr-Commit-Position: refs/heads/master@{#831916}
parent 55e0fcf9
...@@ -2000,6 +2000,7 @@ void CrosNetworkConfig::SetProperties(const std::string& guid, ...@@ -2000,6 +2000,7 @@ void CrosNetworkConfig::SetProperties(const std::string& guid,
<< "SetProperties called with ethernet.eap but no EAP config: " << "SetProperties called with ethernet.eap but no EAP config: "
<< guid; << guid;
std::move(callback).Run(false, kErrorNetworkUnavailable); std::move(callback).Run(false, kErrorNetworkUnavailable);
return;
} }
network = eap_state; network = eap_state;
} }
......
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