diff -r 3568f8f1ccac Misc/ACKS --- a/Misc/ACKS Sun May 19 19:38:12 2013 -0700 +++ b/Misc/ACKS Mon May 20 11:55:37 2013 +0300 @@ -693,6 +693,7 @@ Dom Mitchell Florian Mladitsch Doug Moen +Jaakko Moisio The Dragon De Monsyne Skip Montanaro Paul Moore diff -r 3568f8f1ccac Misc/NEWS --- a/Misc/NEWS Sun May 19 19:38:12 2013 -0700 +++ b/Misc/NEWS Mon May 20 11:55:37 2013 +0300 @@ -12,6 +12,9 @@ - Issue #18019: Fix crash in the repr of dictionaries containing their own views. +- Issue #17976: Fixed potential problem with file_write not detecting IO error + by inspecting the return value of fwrite. + Library ------- diff -r 3568f8f1ccac Objects/fileobject.c --- a/Objects/fileobject.c Sun May 19 19:38:12 2013 -0700 +++ b/Objects/fileobject.c Mon May 20 11:55:37 2013 +0300 @@ -1804,6 +1804,7 @@ const char *s; Py_ssize_t n, n2; PyObject *encoded = NULL; + int err_flag = 0, err; if (f->f_fp == NULL) return err_closed(); @@ -1849,11 +1850,16 @@ FILE_BEGIN_ALLOW_THREADS(f) errno = 0; n2 = fwrite(s, 1, n, f->f_fp); + if (n2 != n || ferror(f->f_fp)) { + err_flag = 1; + err = errno; + } FILE_END_ALLOW_THREADS(f) Py_XDECREF(encoded); if (f->f_binary) PyBuffer_Release(&pbuf); - if (n2 != n) { + if (err_flag) { + errno = err; PyErr_SetFromErrno(PyExc_IOError); clearerr(f->f_fp); return NULL;