diff options
| author | Tao Bao <tbao@google.com> | 2018-07-10 07:56:13 +0200 |
|---|---|---|
| committer | android-build-merger <android-build-merger@google.com> | 2018-07-10 07:56:13 +0200 |
| commit | b5c13bc7ebfe33237141d189b0e8adfb1ecccfaf (patch) | |
| tree | 4136df9997adcb0318b63f2582f10ed2c71541fa | |
| parent | Merge "updater: Remove the support for sha1_check()." am: fbc0f6062c (diff) | |
| parent | Merge "applypatch: Fix a potential nullptr dereferencing." (diff) | |
| download | android_bootable_recovery-b5c13bc7ebfe33237141d189b0e8adfb1ecccfaf.tar android_bootable_recovery-b5c13bc7ebfe33237141d189b0e8adfb1ecccfaf.tar.gz android_bootable_recovery-b5c13bc7ebfe33237141d189b0e8adfb1ecccfaf.tar.bz2 android_bootable_recovery-b5c13bc7ebfe33237141d189b0e8adfb1ecccfaf.tar.lz android_bootable_recovery-b5c13bc7ebfe33237141d189b0e8adfb1ecccfaf.tar.xz android_bootable_recovery-b5c13bc7ebfe33237141d189b0e8adfb1ecccfaf.tar.zst android_bootable_recovery-b5c13bc7ebfe33237141d189b0e8adfb1ecccfaf.zip | |
| -rw-r--r-- | applypatch/applypatch.cpp | 11 |
1 files changed, 7 insertions, 4 deletions
diff --git a/applypatch/applypatch.cpp b/applypatch/applypatch.cpp index e6fd5f6ae..69928bcfb 100644 --- a/applypatch/applypatch.cpp +++ b/applypatch/applypatch.cpp @@ -622,10 +622,13 @@ static int GenerateTarget(const FileContents& source_file, const std::unique_ptr SHA1(reinterpret_cast<const uint8_t*>(patch->data.data()), patch->data.size(), patch_digest); LOG(ERROR) << "patch size " << patch->data.size() << " SHA-1 " << short_sha1(patch_digest); - uint8_t bonus_digest[SHA_DIGEST_LENGTH]; - SHA1(reinterpret_cast<const uint8_t*>(bonus_data->data.data()), bonus_data->data.size(), - bonus_digest); - LOG(ERROR) << "bonus size " << bonus_data->data.size() << " SHA-1 " << short_sha1(bonus_digest); + if (bonus_data != nullptr) { + uint8_t bonus_digest[SHA_DIGEST_LENGTH]; + SHA1(reinterpret_cast<const uint8_t*>(bonus_data->data.data()), bonus_data->data.size(), + bonus_digest); + LOG(ERROR) << "bonus size " << bonus_data->data.size() << " SHA-1 " + << short_sha1(bonus_digest); + } // TODO(b/67849209) Remove after debugging the unit test flakiness. if (android::base::GetMinimumLogSeverity() <= android::base::LogSeverity::DEBUG) { |