This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author vstinner
Recipients Arfrever, georg.brandl, larry, python-dev, richard, serhiy.storchaka, vajrasky, vstinner
Date 2014-02-10.11:10:26
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1392030626.5.0.774242247652.issue20517@psf.upfronthosting.co.za>
In-reply-to
Content
"And you should avoid passing raw bytes string to build an error message, you probably has the Python object version of the filename somewhere in your code."

Oh, I remember the reason why char* must not be used to build an OSError: on Windows, you should never try to decode a bytes filename, because it may raise a UnicodeDecodeError while you are trying to build an OSError. See issue #15478 for the rationale.

Just pass the original PyObject* you get in path_t. There is even an unit test to ensure that OSError.filename is the original name: OSErrorTests.test_oserror_filename() in test_os.
History
Date User Action Args
2014-02-10 11:10:26vstinnersetrecipients: + vstinner, richard, georg.brandl, larry, Arfrever, python-dev, serhiy.storchaka, vajrasky
2014-02-10 11:10:26vstinnersetmessageid: <1392030626.5.0.774242247652.issue20517@psf.upfronthosting.co.za>
2014-02-10 11:10:26vstinnerlinkissue20517 messages
2014-02-10 11:10:26vstinnercreate