Processus index must be 4 bytes (#4455)

secdev · Jul 12, 2024 · 836e4d5 · 836e4d5
1 parent 6b26e21
commit 836e4d5
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 1 deletion.
diff --git a/scapy/utils.py b/scapy/utils.py
@@ -1880,7 +1880,12 @@ def _read_block_epb(self, block, size):
         process_information = {}
         for code, value in options.items():
             if code in [0x8001, 0x8003]:  # PCAPNG_EPB_PIB_INDEX, PCAPNG_EPB_E_PIB_INDEX
-                proc_index = struct.unpack(self.endian + "I", value)[0]
+                try:
+                    proc_index = struct.unpack(self.endian + "I", value)[0]
+                except struct.error:
+                    warning("PcapNg: EPB invalid proc index"
+                            "(expected 4 bytes, got %d) !" % len(value))
+                    raise EOFError
                 if proc_index < len(self.process_information):
                     key = "proc" if code == 0x8001 else "eproc"
                     process_information[key] = self.process_information[proc_index]

diff --git a/test/regression.uts b/test/regression.uts
@@ -2304,13 +2304,18 @@ from io import BytesIO
 # Issue 68352
 file = BytesIO(b"\n\r\r\n\x1c\x00\x00\x00M<+\x1a\x01\x00\x00\x00\xff\xff\xff\xff\xff\xff\xff\xff\x1c\x00\x00\x00\x01\x00\x00\x00\x14\x00\x00\x00\xe4\x00\x00\x00\x00\x00\x04\x00\x14\x00\x00\x00\x01\x00\x00\x00(\x00\x00\x00\xe4\x00\x00\x00\x00\x00\x04\x00\x02\x00\t\x00b'ens16\xb0'\x00\x00\x00\x00\x00\x00\x00(\x00\x00\x00\x06\x00\x00\x004\x00\x00\x00\x01\x00\x00\x00}\x17\x06\x00\xb5t\x1d\x85\x14\x00\x00\x00\x14\x00\x00\x00E\x00\x00\x14\x00\x01\x00\x00@\x00|\xe7\x7f\x00\x00\x01\x7f\x00\x00\x014\x00\x00\x00")
 rdpcap(file)
+
 # Issue 68354
 file = BytesIO(b'\n\r\r\n\xff\xfe\xfe\xffM<+\x1a')
 try:
     rdpcap(file)
 except Scapy_Exception:
     pass
 
+# Issue #70115
+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)
+
 = Read a pcap file with wirelen != captured len
 pktpcapwirelen = rdpcap(pcapwirelenfile)