Skip to content

Commit 03ac445

Browse files
[3.9] gh-134062: Fix hash collisions in IPv4Network and IPv6Network (GH-134063) (GH-134481)
(cherry picked from commit f3fc0c1) Co-authored-by: Mike Salvatore <[email protected]>
1 parent 8d35fd1 commit 03ac445

File tree

3 files changed

+32
-1
lines changed

3 files changed

+32
-1
lines changed

Lib/ipaddress.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -733,7 +733,7 @@ def __eq__(self, other):
733733
return NotImplemented
734734

735735
def __hash__(self):
736-
return hash(int(self.network_address) ^ int(self.netmask))
736+
return hash((int(self.network_address), int(self.netmask)))
737737

738738
def __contains__(self, other):
739739
# always false if one is v4 and the other is v6.

Lib/test/test_ipaddress.py

Lines changed: 28 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2679,6 +2679,34 @@ def testV6HashIsNotConstant(self):
26792679
ipv6_address2 = ipaddress.IPv6Interface("2001:658:22a:cafe:200:0:0:2")
26802680
self.assertNotEqual(ipv6_address1.__hash__(), ipv6_address2.__hash__())
26812681

2682+
# issue 134062 Hash collisions in IPv4Network and IPv6Network
2683+
def testNetworkV4HashCollisions(self):
2684+
self.assertNotEqual(
2685+
ipaddress.IPv4Network("192.168.1.255/32").__hash__(),
2686+
ipaddress.IPv4Network("192.168.1.0/24").__hash__()
2687+
)
2688+
self.assertNotEqual(
2689+
ipaddress.IPv4Network("172.24.255.0/24").__hash__(),
2690+
ipaddress.IPv4Network("172.24.0.0/16").__hash__()
2691+
)
2692+
self.assertNotEqual(
2693+
ipaddress.IPv4Network("192.168.1.87/32").__hash__(),
2694+
ipaddress.IPv4Network("192.168.1.86/31").__hash__()
2695+
)
2696+
2697+
# issue 134062 Hash collisions in IPv4Network and IPv6Network
2698+
def testNetworkV6HashCollisions(self):
2699+
self.assertNotEqual(
2700+
ipaddress.IPv6Network("fe80::/64").__hash__(),
2701+
ipaddress.IPv6Network("fe80::ffff:ffff:ffff:0/112").__hash__()
2702+
)
2703+
self.assertNotEqual(
2704+
ipaddress.IPv4Network("10.0.0.0/8").__hash__(),
2705+
ipaddress.IPv6Network(
2706+
"ffff:ffff:ffff:ffff:ffff:ffff:aff:0/112"
2707+
).__hash__()
2708+
)
2709+
26822710

26832711
if __name__ == '__main__':
26842712
unittest.main()
Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,3 @@
1+
:mod:`ipaddress`: fix collisions in :meth:`~object.__hash__` for
2+
:class:`~ipaddress.IPv4Network` and :class:`~ipaddress.IPv6Network`
3+
objects.

0 commit comments

Comments
 (0)