aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorAndroid Build Coastguard Worker <android-build-coastguard-worker@google.com>2022-04-28 15:58:56 +0000
committerAndroid Build Coastguard Worker <android-build-coastguard-worker@google.com>2022-04-28 15:58:56 +0000
commit4d5812390161ce65ee498b3bdd219489064d032f (patch)
tree16a3fc37ea924893d22684c3e834c4522cdc624b
parent9bd0bd18c388a82a00de0685dcc3d2cd438c8293 (diff)
parent6ff499cf9f327c9503ab028c1ffb60fdd4f7b61a (diff)
downloadpuffin-android13-frc-resolv-release.tar.gz
Snap for 8512216 from 6ff499cf9f327c9503ab028c1ffb60fdd4f7b61a to tm-frc-resolv-releaset_frc_res_330443000android13-frc-resolv-release
Change-Id: Iaae332cd7e47e633045b5b34f2e937739b805c45
-rw-r--r--TEST_MAPPING5
-rw-r--r--src/utils.cc6
2 files changed, 8 insertions, 3 deletions
diff --git a/TEST_MAPPING b/TEST_MAPPING
index ee8f0ca..d78c7bd 100644
--- a/TEST_MAPPING
+++ b/TEST_MAPPING
@@ -3,5 +3,10 @@
{
"name": "puffin_unittest"
}
+ ],
+ "hwasan-postsubmit": [
+ {
+ "name": "puffin_unittest"
+ }
]
}
diff --git a/src/utils.cc b/src/utils.cc
index b2666d3..23d8479 100644
--- a/src/utils.cc
+++ b/src/utils.cc
@@ -185,7 +185,7 @@ bool IsValidGzipHeader(const uint8_t* header, size_t size) {
// 0 1 0x1F
// 1 1 0x8B
// 2 1 compression method (8 denotes deflate)
- static const uint8_t magic[] = {0x1F, 0x8B, 8};
+ static constexpr uint8_t magic[] = {0x1F, 0x8B, 8};
return size >= 10 && std::equal(std::begin(magic), std::end(magic), header);
}
} // namespace
@@ -240,10 +240,10 @@ bool LocateDeflatesInGzip(const Buffer& data, vector<BitExtent>* deflates) {
offset += compressed_size;
// Ignore CRC32 and uncompressed size.
- TEST_AND_RETURN_FALSE(offset + 8 <= data.size());
offset += 8;
member_start = offset;
- } while (IsValidGzipHeader(&data[member_start], data.size() - member_start));
+ } while (member_start < data.size() &&
+ IsValidGzipHeader(&data[member_start], data.size() - member_start));
return true;
}