diff options
| author | Miss Islington (bot) <31488909+miss-islington@users.noreply.github.com> | 2020-06-29 18:12:50 (GMT) |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2020-06-29 18:12:50 (GMT) |
| commit | dc8ce8ead182de46584cc1ed8a8c51d48240cbd5 (patch) | |
| tree | 1db73719f1165dddb6c97a759fdd45bb3f776e49 /Lib/ipaddress.py | |
| parent | 7731139b7af655b9f5df6d1b5493f8dfdf41d569 (diff) | |
| download | cpython-dc8ce8ead182de46584cc1ed8a8c51d48240cbd5.zip cpython-dc8ce8ead182de46584cc1ed8a8c51d48240cbd5.tar.gz cpython-dc8ce8ead182de46584cc1ed8a8c51d48240cbd5.tar.bz2 | |
bpo-41004: Resolve hash collisions for IPv4Interface and IPv6Interface (GH-21033)
The __hash__() methods of classes IPv4Interface and IPv6Interface had issue
of generating constant hash values of 32 and 128 respectively causing hash collisions.
The fix uses the hash() function to generate hash values for the objects
instead of XOR operation
(cherry picked from commit b30ee26e366bf509b7538d79bfec6c6d38d53f28)
Co-authored-by: Ravi Teja P <rvteja92@gmail.com>
Diffstat (limited to 'Lib/ipaddress.py')
| -rw-r--r-- | Lib/ipaddress.py | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/Lib/ipaddress.py b/Lib/ipaddress.py index 873c764..a3a04f7 100644 --- a/Lib/ipaddress.py +++ b/Lib/ipaddress.py @@ -1370,7 +1370,7 @@ class IPv4Interface(IPv4Address): return False def __hash__(self): - return self._ip ^ self._prefixlen ^ int(self.network.network_address) + return hash((self._ip, self._prefixlen, int(self.network.network_address))) __reduce__ = _IPAddressBase.__reduce__ @@ -2017,7 +2017,7 @@ class IPv6Interface(IPv6Address): return False def __hash__(self): - return self._ip ^ self._prefixlen ^ int(self.network.network_address) + return hash((self._ip, self._prefixlen, int(self.network.network_address))) __reduce__ = _IPAddressBase.__reduce__ |