Richard Oudkerk <shibt...@gmail.com> added the comment:

> TBH I don't understand why it should crash, and therefore how your patch 
> helps.  Trying again using narrow strings should always work; indeed, the 
> code did that before I touched it.  Can you describe how it crashes?

The important part of the patch is the removal of the "!" in 

    if (!utime_read_time_arguments(&ua)) {

Without that change, if utime_read_time_arguments(&ua) fails then the unicode 
path is wrongly chosen.  Then PyUnicode_AsUnicode(upath) is called when upath 
has not been initialized.

----------

_______________________________________
Python tracker <rep...@bugs.python.org>
<http://bugs.python.org/issue14127>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

Reply via email to