commit 6f4ef1b4ceed24a273ba317da0d34ab1481fcc0d
author Armando Montanez <amontanez@google.com> Wed Feb 12 10:14:25 2020 -0800
committer CQ Bot Account <commit-bot@chromium.org> Wed Feb 12 19:57:45 2020 +0000
tree 0fcfd7947ea7d718a3dda03c70f4520e2fba2a6e
parent 0743cc58427a187f926808d008e9cf42834f92fe

pw_kvs: Fix checksum finish usage

When KVS calculates a checksum, it does not always call finish. This
change ensures finish is always called when calculating a checksum.

Change-Id: I4898735835e5838c8a5a98e5f738f07346b1059f

pw_kvs/format.cc

diff --git a/pw_kvs/format.cc b/pw_kvs/format.cc
index 44abc6f..cb37591 100644
--- a/pw_kvs/format.cc
+++ b/pw_kvs/format.cc
@@ -53,6 +53,7 @@
     return checksum() == kNoChecksum ? Status::OK : Status::DATA_LOSS;
   }
   CalculateChecksum(algorithm, key, value);
+  algorithm->Finish();
   return algorithm->Verify(checksum_bytes());
 }
 
@@ -103,6 +104,7 @@
     address += read_size;
     bytes_to_read -= read_size;
   }
+  algorithm->Finish();
 
   return algorithm->Verify(checksum_bytes());
 }