r22240 - gnucash/branches/2.4/src/app-utils - Fix a double free crash when the required features dialog is shown
John Ralls
jralls at ceridwen.us
Tue Jun 26 13:28:31 EDT 2012
> From: Geert Janssens <gjanssens at code.gnucash.org>
> To: gnucash-patches at gnucash.org
> Subject: r22240 - gnucash/branches/2.4/src/app-utils - Fix a double
> free crash when the required features dialog is shown
> Message-ID: <201206252054.q5PKsuc9008785 at code.gnucash.org>
> Content-Type: text/plain; charset=UTF-8
>
> Author: gjanssens
> Date: 2012-06-25 16:54:55 -0400 (Mon, 25 Jun 2012)
> New Revision: 22240
> Trac: http://svn.gnucash.org/trac/changeset/22240
>
> Modified:
> gnucash/branches/2.4/src/app-utils/gnc-features.c
> Log:
> Fix a double free crash when the required features dialog is shown
>
Geert,
This doesn't make sense to me. msg is allocated with a starting value, then the relevant feature names are added one at a time in the for loop, allocating a new string each time. Where is msg getting double-freed? ISTM your change will leak all of the strings except the last one.
Regards,
John Ralls
More information about the gnucash-devel
mailing list