Issue 16584: in filecomp._cmp, catch IOError as well as os.error.

Patch by Till Maas.

Issue 16584: in filecomp._cmp, catch IOError as well as os.error.
Patch by Till Maas.
673770c5 · Terry Jan Reedy · db03e6b8 · 673770c5 · 673770c5 · 673770c5
Commit 673770c5 authored May 08, 2013 by Terry Jan Reedy
Hide whitespace changes
Inline Side-by-side

Showing with 5 additions and 1 deletion

Lib/filecmp.py Lib/filecmp.py +1 -1

Misc/ACKS Misc/ACKS +1 -0

Misc/NEWS Misc/NEWS +3 -0

No files found.
--- a/Lib/filecmp.py
+++ b/Lib/filecmp.py
@@ -268,7 +268,7 @@ def cmpfiles(a, b, common, shallow=1):
 def _cmp(a, b, sh, abs=abs, cmp=cmp):
    try:
        return not abs(cmp(a, b, sh))
-    except os.error:
+    except (os.error, IOError):
        return 2



--- a/Misc/ACKS
+++ b/Misc/ACKS
@@ -626,6 +626,7 @@ Mark Lutz
 Jim Lynch
 Mikael Lyngvig
 Martin von Löwis
+Till Maas
 Jeff MacDonald
 John Machin
 Andrew I MacIntyre

--- a/Misc/NEWS
+++ b/Misc/NEWS
@@ -26,6 +26,9 @@ Core and Builtins
 Library
 -------

+- Issue 16584: in filecomp._cmp, catch IOError as well as os.error.
+  Patch by Till Maas.
+
 - Issue #17926: Fix dbm.__contains__ on 64-bit big-endian machines.

 - Issue #17918: When using SSLSocket.accept(), if the SSL handshake failed