From 57cce28b0465282b2944983e1df4d30eafb9ee18 Mon Sep 17 00:00:00 2001
From: Guillaume Valadon <guillaume@valadon.net>
Date: Thu, 5 Sep 2024 20:01:45 +0200
Subject: [PATCH] Check LDAP fields

---
 scapy/layers/ldap.py | 2 ++
 test/regression.uts  | 5 +++++
 2 files changed, 7 insertions(+)

diff --git a/scapy/layers/ldap.py b/scapy/layers/ldap.py
index 494809a6ab7..26db10ce838 100644
--- a/scapy/layers/ldap.py
+++ b/scapy/layers/ldap.py
@@ -729,6 +729,8 @@ def answers(self, other):
         return isinstance(other, LDAP) and other.messageID == self.messageID
 
     def mysummary(self):
+        if not self.protocolOp or not self.messageID:
+            return ""
         return (
             "%s(%s)"
             % (
diff --git a/test/regression.uts b/test/regression.uts
index e182f92b96e..09296a0b141 100644
--- a/test/regression.uts
+++ b/test/regression.uts
@@ -2338,6 +2338,11 @@ except Scapy_Exception:
 file = BytesIO(b"\n\r\r\n\x00\x00\x008\x1a+<M\x00\x01\x00\x00\xef\xff\xff\x81\x01\x00\x00\x01\x00\x00\x00\x0c\x00\x00\x00\x00\x00\x00\x00\x009\x00\x00\x00\x01\x00\x00\x000\x00e\x00\x00\x00\x00\x00\x00\x00\x008\x00\x00\x00\x06\x00\x00\x00d\x00\x00\x00\x00'\x00\x00\x00\x01\x00\x00\x08\x00\x00\x00\x00\x00\x00\x00\x00\x80\x01\x00\x00\x000\x00\x00\x00\x01\x00\x00\x008\x00\x00\n\x0c\x00A\x05\xc0\x01\x00\x00\x00\x00\x00d\x00\x00\x00\x00\x00\x00\x00\x00\x000\x00\x00\x00\x01\x00\x00\x008\x00\x00\n\x0c\x00A\x05\xc0\x83\x00\x00\x043\x01\x00\x00\x00\x00\x00d\x00\x00\x00\x00d")
 l = rdpcap(file)
 
+# Issue #69628
+file = BytesIO(b"M<\xb2\xa1 \x00\xff\xf9\xff\x00\x00\xff\xff\xff\xa1\xb2<M\x01\x00\x00\x00\x00\x00\x00\x17\xca\xca\xca\xca\xca\xca\xca\xca\xca\xca\r\x80\x01[\x00\x01noenum\x00\x00\x00\x00\x00\x06value\x01\x00\x00\x00\x00\x00\x00\x01\x85\xff\xff\x00\x00\x94\xc3\x000\x00\x08\xa0\xf30\x800\x000\x00")
+l = rdpcap(file)
+assert l[0][LDAP].summary() == "LDAP"
+
 = Read a pcap file with wirelen != captured len
 pktpcapwirelen = rdpcap(pcapwirelenfile)