Commit 4798ea25 authored by Maksim Ivanov's avatar Maksim Ivanov Committed by Commit Bot

Fix use-after-move in //chrome/browser/nearby_sharing/logging/

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

Bug: 1122844
Change-Id: I35684eca30719b00c226ab5d27d1d909649802b8
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2382831Reviewed-by: default avatarJames Vecore <vecore@google.com>
Commit-Queue: Maksim Ivanov <emaxx@chromium.org>
Cr-Commit-Position: refs/heads/master@{#802929}
parent 527aea87
......@@ -208,7 +208,8 @@ base::Value DeviceToReadableDictionary(
dict.SetKey("contacts", std::move(contacts_list));
base::Value public_certificates_list(base::Value::Type::LIST);
for (const auto& certificate : device.public_certificates()) {
contacts_list.Append(PublicCertificateToReadableDictionary(certificate));
public_certificates_list.Append(
PublicCertificateToReadableDictionary(certificate));
}
dict.SetKey("public_certificates", std::move(public_certificates_list));
return dict;
......
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