[Pkg-utopia-commits] r1538 - in /packages/unstable/hal/debian: changelog patches/70-libhal-changeset-memleak.patch

[sjoerd at users.alioth.debian.org]

Author: sjoerd
Date: Thu Jun 21 13:20:59 2007
New Revision: 1538

URL: http://svn.debian.org/wsvn/pkg-utopia/?sc=1&rev=1538
Log:
* debian/patches/70-libhal-changeset-memleak.patch
  + Added. Fixes a memleak when freeing hal changesets. (Closes: #429823)

Added:
    packages/unstable/hal/debian/patches/70-libhal-changeset-memleak.patch
Modified:
    packages/unstable/hal/debian/changelog

Modified: packages/unstable/hal/debian/changelog
URL: http://svn.debian.org/wsvn/pkg-utopia/packages/unstable/hal/debian/changelog?rev=1538&op=diff
==============================================================================
--- packages/unstable/hal/debian/changelog (original)
+++ packages/unstable/hal/debian/changelog Thu Jun 21 13:20:59 2007
@@ -3,8 +3,10 @@
   * debian/control
     + Add libsmbios-dev build-depend on amd64 and i386 to build the dell
       backlight addon (Closes: #429871)
-
- -- Sjoerd Simons <sjoerd at debian.org>  Thu, 21 Jun 2007 13:04:54 +0200
+  * debian/patches/70-libhal-changeset-memleak.patch
+    + Added. Fixes a memleak when freeing hal changesets. (Closes: #429823)
+
+ -- Sjoerd Simons <sjoerd at debian.org>  Thu, 21 Jun 2007 14:02:40 +0100
 
 hal (0.5.9.1-1) unstable; urgency=low
 

Added: packages/unstable/hal/debian/patches/70-libhal-changeset-memleak.patch
URL: http://svn.debian.org/wsvn/pkg-utopia/packages/unstable/hal/debian/patches/70-libhal-changeset-memleak.patch?rev=1538&op=file
==============================================================================
--- packages/unstable/hal/debian/patches/70-libhal-changeset-memleak.patch (added)
+++ packages/unstable/hal/debian/patches/70-libhal-changeset-memleak.patch Thu Jun 21 13:20:59 2007
@@ -1,0 +1,21 @@
+commit 314ee3c92e02b35c19a14e0d7465e7c0df2a7903
+Author: Sjoerd Simons <sjoerd at luon.net>
+Date:   Thu Jun 21 14:58:54 2007 +0200
+
+      free changeset elements keys
+    
+      Changeset element keys weren't freed, causing memleaks. Mostly in the UPS
+      addon, which uses it for every change it notices.
+
+diff --git a/libhal/libhal.c b/libhal/libhal.c
+index fb9800a..c8e41aa 100644
+--- a/libhal/libhal.c
++++ b/libhal/libhal.c
+@@ -4191,6 +4191,7 @@ libhal_device_free_changeset (LibHalChangeSet *changeset)
+ 			fprintf (stderr, "%s %d : unknown change_type %d\n", __FILE__, __LINE__, elem->change_type);
+ 			break;
+ 		}
++		free (elem->key);
+ 		free (elem);
+ 	}
+