[Python-Dev] cpython (3.6): replace usage of Py_VA_COPY with the (C99) standard va_copy

Benjamin Peterson benjamin at python.org
Thu Sep 22 02:01:01 EDT 2016



On Wed, Sep 21, 2016, at 02:06, Christian Heimes wrote:
> On 2016-09-21 05:39, benjamin.peterson wrote:
> > https://hg.python.org/cpython/rev/278b21d8e86e
> > changeset:   103977:278b21d8e86e
> > branch:      3.6
> > parent:      103975:d31b4de433b7
> > user:        Benjamin Peterson <benjamin at python.org>
> > date:        Tue Sep 20 20:39:33 2016 -0700
> > summary:
> >   replace usage of Py_VA_COPY with the (C99) standard va_copy
> 
> Thanks! Coverity has been complaining about Py_VA_COPY() for a long
> time. Your change may cause a memory leak on some platforms. You must
> va_end() a va_copy() region:

Yep. Thanks for fixing that. I'm not actually aware of any platform
where va_end() frees anything, but it's the right thing to do.


More information about the Python-Dev mailing list