Bug#507456: Memory leak when freeing a nvp

[Samuel Tardieu]

Package: mairix
Version: 0.21-1
Severity: normal
Tags: patch

There is a leak in mairix which makes it not free the nvp_entry structures.

Patch has been submitted upstream.

-- System Information:
Debian Release: lenny/sid
  APT prefers unstable
  APT policy: (990, 'unstable'), (500, 'testing'), (500, 'stable'), (1, 
'experimental')
Architecture: i386 (i686)

Kernel: Linux 2.6.27-rc1
Locale: LANG=fr_FR.UTF-8, LC_CTYPE=fr_FR.UTF-8 (charmap=UTF-8) (ignored: LC_ALL 
set to en_US.UTF-8)
Shell: /bin/sh linked to /bin/bash

Versions of packages mairix depends on:
ii  libbz2-1.0             1.0.5-1           high-quality block-sorting file co
ii  libc6                  2.7-16            GNU C Library: Shared libraries
ii  zlib1g                 1:1.2.3.3.dfsg-12 compression library - runtime

mairix recommends no packages.

mairix suggests no packages.

-- no debconf information

commit df6d7095e5d6a1333fb83fe20944dbfc7f6cdd17
Author: Samuel Tardieu <[EMAIL PROTECTED]>
Date:   Fri Nov 28 17:56:07 2008 +0100

    When freeing a struct nvp, do not forget to free the struct nvp_entry.
    
    This fixes a recurrent memory leak, as the entries were not freed.

diff --git a/nvp.c b/nvp.c
index bb5c6cb..560e7d6 100644
--- a/nvp.c
+++ b/nvp.c
@@ -274,6 +274,7 @@ void free_nvp(struct nvp *nvp)/*{{{*/
         free(ne->rhs);
         break;
     }
+    free(ne);
   }
   free(nvp);
 }