diff --git a/scapy/contrib/openflow.py b/scapy/contrib/openflow.py
index dd11630acfd95fbb7c618996cf18c63631a58aa3..62c0f031d6d3199fef51d5baa9bd52dc7ec25893 100755
--- a/scapy/contrib/openflow.py
+++ b/scapy/contrib/openflow.py
@@ -1183,7 +1183,7 @@ ofpt_cls = {  0: OFPTHello,
 TCP_guess_payload_class_copy = TCP.guess_payload_class
 
 def OpenFlow(self, payload):
-    if self is None or self.dport == 6653 or self.dport == 6633 or self.sport == 6653 or self.sport == 6653:
+    if self is None or self.dport == 6653 or self.dport == 6633 or self.sport == 6653 or self.sport == 6633:
     # port 6653 has been allocated by IANA, port 6633 should no longer be used
     # OpenFlow function may be called with None self in OFPPacketField
         of_type = ord(payload[1])
diff --git a/scapy/contrib/openflow3.py b/scapy/contrib/openflow3.py
index 48f83b4df380ad2e74064569f1539efdf8b3fabd..a7ea3fa66875020b01948918c8adba0987ceb24f 100755
--- a/scapy/contrib/openflow3.py
+++ b/scapy/contrib/openflow3.py
@@ -3348,7 +3348,7 @@ ofpt_cls = {  0: OFPTHello,
 TCP_guess_payload_class_copy = TCP.guess_payload_class
 
 def OpenFlow(self, payload):
-    if self is None or self.dport == 6653 or self.dport == 6633 or self.sport == 6653 or self.sport == 6653:
+    if self is None or self.dport == 6653 or self.dport == 6633 or self.sport == 6653 or self.sport == 6633:
     # port 6653 has been allocated by IANA, port 6633 should no longer be used
     # OpenFlow function may be called with None self in OFPPacketField
         of_type = ord(payload[1])