Re: [patch] Fix [BZ#14122] memory leak in nss_parse_service_list

[Paul Pluzhnikov <ppluzhnikov at google dot com>]

On Sun, May 20, 2012 at 4:22 PM, Paul Pluzhnikov <ppluzhnikov@google.com> wrote:

> Here is a rather simple fix for [BZ#14122]: memory leak in
> nss_parse_service_list.

That patch fixed leaks in the error paths, but did not fix the original problem.

Attached is an updated patch that fixes both.

Re-tested on Linux/x86_64 an i686.

Thanks,
--
Paul Pluzhnikov


2012-05-20  Paul Pluzhnikov  <ppluzhnikov@google.com>

        [BZ #14122]
        * nss/nsswitch.c (defconfig_entries): New variable.
        (__nss_database_lookup): Don't leak defconfig entries.
        (nss_parse_service_list): Don't leak on error paths.
        (free_database_entries): New function.
        (free_defconfig): New function.
        (free_mem): Move common code to free_database_entries.

Attachment: glibc-nsswitch-leak-20120520a.txt
Description: Text document