Commit 108e0b4d authored by Maksim Ivanov's avatar Maksim Ivanov Committed by Commit Bot

Fix use-after-move in //services/network/

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

Bug: 1122844
Change-Id: Ie5df7ae4672bd39945ccce51d163835b254a43fd
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2387020
Commit-Queue: Matt Menke <mmenke@chromium.org>
Reviewed-by: default avatarMatt Menke <mmenke@chromium.org>
Cr-Commit-Position: refs/heads/master@{#804010}
parent 84fe3e5a
...@@ -97,7 +97,8 @@ void SocketFactory::CreateTCPBoundSocket( ...@@ -97,7 +97,8 @@ void SocketFactory::CreateTCPBoundSocket(
std::move(callback).Run(result, base::nullopt); std::move(callback).Run(result, base::nullopt);
return; return;
} }
socket->set_id( TCPBoundSocket* socket_ptr = socket.get();
socket_ptr->set_id(
tcp_bound_socket_receivers_.Add(std::move(socket), std::move(receiver))); tcp_bound_socket_receivers_.Add(std::move(socket), std::move(receiver)));
std::move(callback).Run(result, local_addr_out); std::move(callback).Run(result, local_addr_out);
} }
......
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