public inbox for libc-locales@sourceware.org
 help / color / mirror / Atom feed
* [Bug localedata/18871] New: _nl_load_domain memory leak
@ 2015-08-26  0:39 eggert at gnu dot org
  2015-08-26 12:17 ` [Bug localedata/18871] " joseph at codesourcery dot com
                   ` (4 more replies)
  0 siblings, 5 replies; 6+ messages in thread
From: eggert at gnu dot org @ 2015-08-26  0:39 UTC (permalink / raw)
  To: libc-locales

https://sourceware.org/bugzilla/show_bug.cgi?id=18871

            Bug ID: 18871
           Summary: _nl_load_domain memory leak
           Product: glibc
           Version: 2.22
            Status: NEW
          Severity: normal
          Priority: P2
         Component: localedata
          Assignee: unassigned at sourceware dot org
          Reporter: eggert at gnu dot org
                CC: libc-locales at sourceware dot org
  Target Milestone: ---

Created attachment 8552
  --> https://sourceware.org/bugzilla/attachment.cgi?id=8552&action=edit
Fix memory leak in _nl_load_domain

A Coverity scan found a memory leak in _nl_load_domain.  Obvious patch is
attached.

-- 
You are receiving this mail because:
You are on the CC list for the bug.

^ permalink raw reply	[flat|nested] 6+ messages in thread

* [Bug localedata/18871] _nl_load_domain memory leak
  2015-08-26  0:39 [Bug localedata/18871] New: _nl_load_domain memory leak eggert at gnu dot org
@ 2015-08-26 12:17 ` joseph at codesourcery dot com
  2015-08-26 19:28 ` eggert at gnu dot org
                   ` (3 subsequent siblings)
  4 siblings, 0 replies; 6+ messages in thread
From: joseph at codesourcery dot com @ 2015-08-26 12:17 UTC (permalink / raw)
  To: libc-locales

https://sourceware.org/bugzilla/show_bug.cgi?id=18871

--- Comment #1 from joseph at codesourcery dot com <joseph at codesourcery dot com> ---
This file is shared with GNU gettext, so if the problem is present in 
current gettext sources then the fix should be submitted there as well 
(although we can apply local fixes as needed even if they aren't yet in 
gettext).

-- 
You are receiving this mail because:
You are on the CC list for the bug.

^ permalink raw reply	[flat|nested] 6+ messages in thread

* [Bug localedata/18871] _nl_load_domain memory leak
  2015-08-26  0:39 [Bug localedata/18871] New: _nl_load_domain memory leak eggert at gnu dot org
  2015-08-26 12:17 ` [Bug localedata/18871] " joseph at codesourcery dot com
@ 2015-08-26 19:28 ` eggert at gnu dot org
  2015-08-27 20:50 ` carlos at redhat dot com
                   ` (2 subsequent siblings)
  4 siblings, 0 replies; 6+ messages in thread
From: eggert at gnu dot org @ 2015-08-26 19:28 UTC (permalink / raw)
  To: libc-locales

https://sourceware.org/bugzilla/show_bug.cgi?id=18871

--- Comment #2 from Paul Eggert <eggert at gnu dot org> ---
Thanks for suggesting.  I submitted the fix to the GNU gettext maintainers
here:

http://lists.gnu.org/archive/html/bug-gettext/2015-08/msg00033.html

-- 
You are receiving this mail because:
You are on the CC list for the bug.

^ permalink raw reply	[flat|nested] 6+ messages in thread

* [Bug localedata/18871] _nl_load_domain memory leak
  2015-08-26  0:39 [Bug localedata/18871] New: _nl_load_domain memory leak eggert at gnu dot org
  2015-08-26 12:17 ` [Bug localedata/18871] " joseph at codesourcery dot com
  2015-08-26 19:28 ` eggert at gnu dot org
@ 2015-08-27 20:50 ` carlos at redhat dot com
  2015-08-27 22:01 ` [Bug locale/18871] " jsm28 at gcc dot gnu.org
  2018-12-21 15:24 ` bruno at clisp dot org
  4 siblings, 0 replies; 6+ messages in thread
From: carlos at redhat dot com @ 2015-08-27 20:50 UTC (permalink / raw)
  To: libc-locales

https://sourceware.org/bugzilla/show_bug.cgi?id=18871

Carlos O'Donell <carlos at redhat dot com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |carlos at redhat dot com

--- Comment #3 from Carlos O'Donell <carlos at redhat dot com> ---
This fix looks good to me.

-- 
You are receiving this mail because:
You are on the CC list for the bug.

^ permalink raw reply	[flat|nested] 6+ messages in thread

* [Bug locale/18871] _nl_load_domain memory leak
  2015-08-26  0:39 [Bug localedata/18871] New: _nl_load_domain memory leak eggert at gnu dot org
                   ` (2 preceding siblings ...)
  2015-08-27 20:50 ` carlos at redhat dot com
@ 2015-08-27 22:01 ` jsm28 at gcc dot gnu.org
  2018-12-21 15:24 ` bruno at clisp dot org
  4 siblings, 0 replies; 6+ messages in thread
From: jsm28 at gcc dot gnu.org @ 2015-08-27 22:01 UTC (permalink / raw)
  To: libc-locales

https://sourceware.org/bugzilla/show_bug.cgi?id=18871

Joseph Myers <jsm28 at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
          Component|localedata                  |locale

-- 
You are receiving this mail because:
You are on the CC list for the bug.

^ permalink raw reply	[flat|nested] 6+ messages in thread

* [Bug locale/18871] _nl_load_domain memory leak
  2015-08-26  0:39 [Bug localedata/18871] New: _nl_load_domain memory leak eggert at gnu dot org
                   ` (3 preceding siblings ...)
  2015-08-27 22:01 ` [Bug locale/18871] " jsm28 at gcc dot gnu.org
@ 2018-12-21 15:24 ` bruno at clisp dot org
  4 siblings, 0 replies; 6+ messages in thread
From: bruno at clisp dot org @ 2018-12-21 15:24 UTC (permalink / raw)
  To: libc-locales

https://sourceware.org/bugzilla/show_bug.cgi?id=18871

Bruno Haible <bruno at clisp dot org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |bruno at clisp dot org

--- Comment #4 from Bruno Haible <bruno at clisp dot org> ---
The fix looks good to me as well, as has been in GNU gettext for 3 years now.

Paul's fix can now be applied to glibc as well.

-- 
You are receiving this mail because:
You are on the CC list for the bug.

^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2018-12-21 15:24 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-08-26  0:39 [Bug localedata/18871] New: _nl_load_domain memory leak eggert at gnu dot org
2015-08-26 12:17 ` [Bug localedata/18871] " joseph at codesourcery dot com
2015-08-26 19:28 ` eggert at gnu dot org
2015-08-27 20:50 ` carlos at redhat dot com
2015-08-27 22:01 ` [Bug locale/18871] " jsm28 at gcc dot gnu.org
2018-12-21 15:24 ` bruno at clisp dot org

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).