Commit 1f48e110 authored by Maksim Ivanov's avatar Maksim Ivanov Committed by Commit Bot

Fix use-after-move in //device/fido/fido_device_authenticator.cc

Fix use-after-move (potential) bugs found by the
"bugprone-use-after-move" clang-tidy check.

Bug: 1122844
Change-Id: Iad9f9a167eda67d20d9e99861804a1f2183fe3fd
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2385455Reviewed-by: default avatarAdam Langley <agl@chromium.org>
Commit-Queue: Maksim Ivanov <emaxx@chromium.org>
Cr-Commit-Position: refs/heads/master@{#803212}
parent 028bf31d
......@@ -230,6 +230,7 @@ void FidoDeviceAuthenticator::OnHaveEphemeralKey(
base::Optional<pin::KeyAgreementResponse> key) {
if (status != CtapDeviceResponseCode::kSuccess) {
std::move(callback).Run(status, base::nullopt);
return;
}
DCHECK(key.has_value());
......
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