27 Dec
2010
27 Dec
'10
5:45 p.m.
Benjamin wrote:
On Sun, Dec 26, 2010 at 11:17 PM, corvid <corvid@lavabit.com> wrote:
BTW, I was looking through prefgui... Are you leaking memory in apply()?
I should hope not, but there's a chance -- which lines were you thinking of? I based that code on prefs.c, which does a couple funny-looking things like dStrdup'ing string constants.
In PrefsParser::parseOption(), when a string or url is changed from its initial value, there's a dFree() or a_Url_free() of the previous value. I didn't notice such freeing in prefsgui.