summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorHenning Baldersheim <balder@yahoo-inc.com>2020-10-26 11:11:17 +0100
committerGitHub <noreply@github.com>2020-10-26 11:11:17 +0100
commit88330f8df080a9204aa13b5668e2e9754f6e279d (patch)
tree87d6001442e81b156067e88a9cd70d543a69e0b5
parentdb637ffe7938cee15560cd94f7740c523d18bab4 (diff)
parenta4d8ea7b6d2e66526b837eb8540aae532824802d (diff)
Merge pull request #15032 from vespa-engine/vekterli/avoid-mergehandler-spam-log-level-crash
Don't try to access container element prior to adding it when spam logging
-rw-r--r--storage/src/vespa/storage/persistence/mergehandler.cpp2
1 files changed, 1 insertions, 1 deletions
diff --git a/storage/src/vespa/storage/persistence/mergehandler.cpp b/storage/src/vespa/storage/persistence/mergehandler.cpp
index 4fe7333fb5f..c7c681a838b 100644
--- a/storage/src/vespa/storage/persistence/mergehandler.cpp
+++ b/storage/src/vespa/storage/persistence/mergehandler.cpp
@@ -403,9 +403,9 @@ MergeHandler::fetchLocalData(
|| (entries.empty() && alreadyFilled == 0))
{
remainingSize -= entry->getSize();
+ entries.push_back(std::move(entry));
LOG(spam, "Added %s, remainingSize is %u",
entries.back()->toString().c_str(), remainingSize);
- entries.push_back(std::move(entry));
} else {
LOG(spam, "Adding %s would exceed chunk size limit of %u; "
"not filling up any more diffs for current round",