GNU bug report logs - #35179
[PATCH] Plug memory leak in GTK x-display-monitor-attributes-list

Previous Next

Package: emacs;

Reported by: Alex <agrambot <at> gmail.com>

Date: Sun, 7 Apr 2019 05:18:01 UTC

Severity: normal

Tags: patch

Done: Alex <agrambot <at> gmail.com>

Bug is archived. No further changes may be made.

Full log


Message #29 received at 35179 <at> debbugs.gnu.org (full text, mbox):

From: Eli Zaretskii <eliz <at> gnu.org>
To: Alex <agrambot <at> gmail.com>
Cc: 35179 <at> debbugs.gnu.org
Subject: Re: bug#35179: [PATCH] Plug memory leak in
 GTK	x-display-monitor-attributes-list
Date: Sun, 07 Apr 2019 21:21:20 +0300
> From: Alex <agrambot <at> gmail.com>
> Cc: 35179 <at> debbugs.gnu.org
> Date: Sun, 07 Apr 2019 11:44:54 -0600
> 
> Whoops, I forgot to add a closing parenthesis...

Thanks.  This variant is fine with me, but since we use xfree to free
the name, I think we should use xstrdup, not g_strdup, to copy it, as
the latter is documented to need g_free to free the storage.




This bug report was last modified 6 years and 46 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.