Author serhiy.storchaka
Recipients brett.cannon, serhiy.storchaka, steve.dower, vstinner, xiang.zhang
Date 2016-12-21.15:32:54
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1482334374.98.0.494123857202.issue29034@psf.upfronthosting.co.za>
In-reply-to
Content
There are many ways to write a solution of the original issue. You can just add a number of "Py_DECREF(bytes)" (path_converter.patch), or add new variable to_cleanup2 (path_converter-v2.patch), or reuse to_cleanup (path_converter-to_cleanup.patch), or clean ups bytes (path_converter-bytes.patch). All these ways look equivalent.

The issue mentioned in msg283754 is more severe. It can be solved by making path->object referring an original argument (but some code checks the type of path->object), or making path->object owning a reference.

It seems to me that path->narrow can be not initialized for str path on Windows.
History
Date User Action Args
2016-12-21 15:32:55serhiy.storchakasetrecipients: + serhiy.storchaka, brett.cannon, vstinner, steve.dower, xiang.zhang
2016-12-21 15:32:54serhiy.storchakasetmessageid: <1482334374.98.0.494123857202.issue29034@psf.upfronthosting.co.za>
2016-12-21 15:32:54serhiy.storchakalinkissue29034 messages
2016-12-21 15:32:54serhiy.storchakacreate