Eric V. Smith <e...@trueblade.com> added the comment:

Despite the fact that as of now (or 6+ years ago!) the only way to trigger the 
malloc() is via VT_LPSTR, I still think the way the free() call is written is 
bad. What if another type is added?

If that were fixed, I still think the basic idea of this patch in isolation of 
the other raised issues is a good idea. If it was converted to a PR, I'd 
support merging it (although I'd like to hear from the Windows folks).

But it is curious that this bug is so old and it hasn't caused more problems.

----------

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

Reply via email to