Copy digest before emplacing in BlobContainer

...fixing potentially dangerous code (evaluation order is unspecified).

1 files changed, 3 insertions, 2 deletions

diff --git a/src/buildtool/execution_api/bazel_msg/bazel_blob_container.hpp b/src/buildtool/execution_api/bazel_msg/bazel_blob_container.hpp
index 2e028d05..41f30ddd 100644
--- a/src/buildtool/execution_api/bazel_msg/bazel_blob_container.hpp
+++ b/src/buildtool/execution_api/bazel_msg/bazel_blob_container.hpp
@@ -230,13 +230,14 @@ class BlobContainer {
     explicit BlobContainer(std::vector<BazelBlob> blobs) {
         blobs_.reserve(blobs.size());
         for (auto& blob : blobs) {
-            blobs_.emplace(blob.digest, std::move(blob));
+            this->Emplace(std::move(blob));
         }
     }
 
     /// \brief Emplace new BazelBlob to container.
     void Emplace(BazelBlob&& blob) {
-        blobs_.emplace(blob.digest, std::move(blob));
+        bazel_re::Digest digest = blob.digest;
+        blobs_.emplace(std::move(digest), std::move(blob));
     }
 
     /// \brief Clear all BazelBlobs from container.