diff options
author | Daniel Zheng <zhengdaniel@google.com> | 2024-01-29 12:13:00 -0800 |
---|---|---|
committer | Daniel Zheng <zhengdaniel@google.com> | 2024-01-30 09:24:24 -0800 |
commit | 1e56e35301076675b702e933f97edcbacd5046f8 (patch) | |
tree | f819977f9f1935c98c3d5370bc0987837720807f | |
parent | b06aeb7f14dd23650b7592b96244e0f62a3bc8b3 (diff) | |
download | update_engine-1e56e35301076675b702e933f97edcbacd5046f8.tar.gz |
update_engine: add error log
In cases error pointer is unitialized, write method might just fail
without a log. Adding this here in case for easier debugging
Test: th
Change-Id: Ide17326eea5ab5ec620e1246c95df80c7bc6808c
-rw-r--r-- | payload_consumer/delta_performer.cc | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/payload_consumer/delta_performer.cc b/payload_consumer/delta_performer.cc index b4a3970c..8083cb19 100644 --- a/payload_consumer/delta_performer.cc +++ b/payload_consumer/delta_performer.cc @@ -322,8 +322,8 @@ MetadataParseResult DeltaPerformer::ParsePayloadMetadata( if (metadata_size_ + metadata_signature_size_ > payload_->size) { LOG(ERROR) << "The size of the metadata_size(" << metadata_size_ << ")" << " or metadata signature(" << metadata_signature_size_ << ")" - << " is greater than the size of the payload" - << "(" << payload_->size << ")"; + << " is greater than the size of the payload" << "(" + << payload_->size << ")"; *error = ErrorCode::kDownloadInvalidMetadataSize; return MetadataParseResult::kError; } @@ -440,6 +440,7 @@ bool DeltaPerformer::CheckSPLDowngrade() { // and stores an action exit code in |error|. bool DeltaPerformer::Write(const void* bytes, size_t count, ErrorCode* error) { if (!error) { + LOG(INFO) << "Error Code is not initialized"; return false; } *error = ErrorCode::kSuccess; |