diff --git a/qubesmanager/firewall.py b/qubesmanager/firewall.py
index d8a3d1c..14b69dc 100644
--- a/qubesmanager/firewall.py
+++ b/qubesmanager/firewall.py
@@ -244,7 +244,7 @@ class QubesFirewallRulesModel(QtCore.QAbstractItemModel):
                 allow_dns = True
                 continue
 
-            if rule.proto == icmp_rule:
+            if rule == icmp_rule:
                 allow_icmp = True
                 continue