Commit c5edae40 authored by Maksim Ivanov's avatar Maksim Ivanov Committed by Commit Bot

Fix use-after-move in //content/browser/data_url_loader_factory.cc

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

Bug: 1122844
Change-Id: I7c78266d7455037d4262d6c272fed1dfcd89df8b
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2382833Reviewed-by: default avatarJohn Abd-El-Malek <jam@chromium.org>
Commit-Queue: Maksim Ivanov <emaxx@chromium.org>
Cr-Commit-Position: refs/heads/master@{#803141}
parent 8f2cf7fa
......@@ -91,9 +91,10 @@ void DataURLLoaderFactory::CreateLoaderAndStart(
write_data->producer =
std::make_unique<mojo::DataPipeProducer>(std::move(producer));
mojo::DataPipeProducer* producer_ptr = write_data->producer.get();
base::StringPiece string_piece(write_data->data);
write_data->producer->Write(
producer_ptr->Write(
std::make_unique<mojo::StringDataSource>(
string_piece, mojo::StringDataSource::AsyncWritingMode::
STRING_STAYS_VALID_UNTIL_COMPLETION),
......
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