Skip to content

Commit 1982463

Browse files
Zainullin DamirZainullin Damir
authored andcommitted
++ tls parser bugfix
1 parent f7b862c commit 1982463

File tree

1 file changed

+4
-3
lines changed

1 file changed

+4
-3
lines changed

src/plugins/process/common/tlsParser/tlsParser.cpp

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -239,6 +239,10 @@ bool TLSParser::parse(std::span<const std::byte> payload, const bool isQUIC) noe
239239
return false;
240240
}
241241

242+
if (compressionMethodsOffset + *compressionMethodsLength > payload.size()) {
243+
return false;
244+
}
245+
242246
m_extensions = getExtensionsSection(
243247
payload.subspan(compressionMethodsOffset + *compressionMethodsLength));
244248
if (!m_extensions.has_value()) {
@@ -406,9 +410,6 @@ std::optional<TLSParser::SupportedVersions> TLSParser::parseSupportedVersions(
406410
if (sizeof(uint8_t) + versionsLength > extension.size()) {
407411
return std::nullopt;
408412
}
409-
if (versionsLength % sizeof(uint16_t) != 0) {
410-
return std::nullopt;
411-
}
412413

413414
std::ranges::copy(
414415
toSpan<const uint16_t>(

0 commit comments

Comments
 (0)