Handle exception of getLdapOperationAsn1Record() in `getLdapOperati…

…onType()` (#1617)
seladb · Oct 23, 2024 · d3adb80 · d3adb80
1 parent 22f48d2
commit d3adb80
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 1 deletion.
diff --git a/Packet++/src/LdapLayer.cpp b/Packet++/src/LdapLayer.cpp
@@ -343,7 +343,17 @@ namespace pcpp
 
 	LdapOperationType LdapLayer::getLdapOperationType() const
 	{
-		return LdapOperationType::fromUintValue(getLdapOperationAsn1Record()->getTagType());
+		uint8_t tagType;
+		try
+		{
+			tagType = getLdapOperationAsn1Record()->getTagType();
+		}
+		catch (...)
+		{
+			tagType = LdapOperationType::Unknown;
+		}
+
+		return LdapOperationType::fromUintValue(tagType);
 	}
 
 	void LdapLayer::parseNextLayer()

diff --git a/Tests/Fuzzers/RegressionTests/regression_samples/crash-exception-cast-constructed-record b/Tests/Fuzzers/RegressionTests/regression_samples/crash-exception-cast-constructed-record