Re: strippwd patch against CVS20041108
- From: Jindrich Novy <jnovy redhat com>
- To: "Andrew V. Samoilov" <andrew email zp ua>
- Cc: MC Devel <mc-devel gnome org>
- Subject: Re: strippwd patch against CVS20041108
- Date: Wed, 10 Nov 2004 18:39:54 +0100
Hello Andrew,
On Wed, 2004-11-10 at 11:51, Andrew V. Samoilov wrote:
> Hello,
>
> > LdO> These are the hunks of the strippwd patch that have not yet been
> > LdO> committed.
> >
> > Committed with my fixes.
>
> Patch below fixes double free, memory leaks and missed {}.
> Also it introduces new enum to do add2hotlist() calls more readable.
>
> Second part is optimization.
About the (add2hotlist_cmd): Eliminate static label_string:
I implemented the label_string to be static in order to your comment #16
on Red Hat bugzilla:
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=131088
where you replied to my question where the label_string is leaked in my
original implementation of strippwd patch (where label_string is
g_strdup()ed):
AVS >label_string is leaked if input_dialog() returns NULL or empty
string.
AVS >In last case this memory (g_strdup("")) is also leaked.
>From the patch I see you reintroduced this. Otherwise please make it
clear to me.
--
Jindrich Novy <jnovy redhat com>, http://people.redhat.com/jnovy/
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]