diff options
| author |   Yaroslav Brustinov <ybrustin@cisco.com> | 2017-03-15 15:28:30 +0200 |
|---|---|---|
| committer | Yaroslav Brustinov <ybrustin@cisco.com> | 2017-03-15 15:28:30 +0200 |
| commit | f9b01d2d64a2902770a2d929cf4fbe1d83fa9588 (patch) | |
| tree | 1069c3391f063d8b0c496bf5b1bee732fb0e3c8b | |
| parent | 051360159df9fddc6c9e99a032780247d4b7c9db (diff) | |
Verify Scapy build does not ruin raw data
Change-Id: I3792dddad68e62f13c359aede7ded8292f9a92a5
Signed-off-by: Yaroslav Brustinov <ybrustin@cisco.com>
4 files changed, 6 insertions, 8 deletions
diff --git a/scripts/external_libs/scapy-2.3.1/python2/scapy/fields.py b/scripts/external_libs/scapy-2.3.1/python2/scapy/fields.py index ba65e637..80716700 100644 --- a/scripts/external_libs/scapy-2.3.1/python2/scapy/fields.py +++ b/scripts/external_libs/scapy-2.3.1/python2/scapy/fields.py @@ -34,7 +34,7 @@ class Field: self.default = self.any2i(None,default) self.sz = struct.calcsize(self.fmt) self.owners = [] - self._offset =0; + self._offset = 0 def get_size_bytes (self): if hasattr(self, 'size'): diff --git a/scripts/external_libs/scapy-2.3.1/python2/scapy/packet.py b/scripts/external_libs/scapy-2.3.1/python2/scapy/packet.py index 8cd78b86..b22305bb 100644 --- a/scripts/external_libs/scapy-2.3.1/python2/scapy/packet.py +++ b/scripts/external_libs/scapy-2.3.1/python2/scapy/packet.py @@ -320,8 +320,6 @@ class Packet(BasePacket): def self_build(self, field_pos_list=None): - #if self.raw_packet_cache is not None: - # return self.raw_packet_cache p="" for f in self.fields_desc: if type(p) is tuple : @@ -339,7 +337,8 @@ class Packet(BasePacket): f._offset= val else: p = f.addfield(self, p, val) - + if self.raw_packet_cache is not None: + assert p == self.raw_packet_cache, 'Could not build the packet.' return p def do_build_payload(self): diff --git a/scripts/external_libs/scapy-2.3.1/python3/scapy/fields.py b/scripts/external_libs/scapy-2.3.1/python3/scapy/fields.py index 9dcebcf3..a9e66137 100644 --- a/scripts/external_libs/scapy-2.3.1/python3/scapy/fields.py +++ b/scripts/external_libs/scapy-2.3.1/python3/scapy/fields.py @@ -34,8 +34,7 @@ class Field: self.default = self.any2i(None,default) self.sz = struct.calcsize(self.fmt) self.owners = [] - self.offset =0; - self._offset =0 + self._offset = 0 def get_size_bytes (self): diff --git a/scripts/external_libs/scapy-2.3.1/python3/scapy/packet.py b/scripts/external_libs/scapy-2.3.1/python3/scapy/packet.py index 7e0a0898..8401e0cf 100644 --- a/scripts/external_libs/scapy-2.3.1/python3/scapy/packet.py +++ b/scripts/external_libs/scapy-2.3.1/python3/scapy/packet.py @@ -338,8 +338,6 @@ class Packet(BasePacket, metaclass = Packet_metaclass): print ("field %-40s %02d %02d" % (f.name, f._offset,f.get_size_bytes () ) ); def self_build(self, field_pos_list=None): - #if self.raw_packet_cache is not None: - # return self.raw_packet_cache p=b"" for f in self.fields_desc: #print(f.name) @@ -359,6 +357,8 @@ class Packet(BasePacket, metaclass = Packet_metaclass): f._offset= val else: p = f.addfield(self, p, val) + if self.raw_packet_cache is not None: + assert p == self.raw_packet_cache, 'Could not build the packet.' return p def do_build_payload(self): |