Merge pull request #1268 from guedou/Issue_#1263

Ack Policy field renamed

Author: guedou

scapy/layers/dot11.py

@@ -180,7 +180,7 @@ def unwep(self, key=None, warn=1):
 class Dot11QoS(Packet):
     name = "802.11 QoS"
     fields_desc = [ BitField("Reserved",None,1),
-                    BitField("Ack Policy",None,2),
+                    BitField("Ack_Policy", None, 2),
                     BitField("EOSP",None,1),
                     BitField("TID",None,4),
                     ByteField("TXOP",None) ]

test/regression.uts

@@ -9466,8 +9466,11 @@ Dot11 in p and p.addr3 == "00:00:00:00:00:00"
 p.mysummary() == '802.11 Management 0 00:00:00:00:00:00 > 00:00:00:00:00:00'
 
 = Dot11QoS - build
+s = raw(Dot11()/Dot11QoS(Ack_Policy=1))
+assert s == b'\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00 \x00'
+
 s = raw(Dot11(type=2, subtype=8)/Dot11QoS(TID=4))
-s == b'\x88\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x04\x00'
+assert s == b'\x88\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x04\x00'
 
 = Dot11 - binary in SSID
 pkt = Dot11() / Dot11Beacon() / Dot11Elt(ID=0, info=b"".join(chb(i) for i in range(32)))