rpms/gkrellm/devel gkrellm-2.3.1-leak-fix.patch, NONE, 1.1 gkrellm.spec, 1.17, 1.18

[Hans de Goede]

Author: jwrdegoede

Update of /cvs/extras/rpms/gkrellm/devel
In directory cvs1.fedora.phx.redhat.com:/tmp/cvs-serv21580

Modified Files:
	gkrellm.spec 
Added Files:
	gkrellm-2.3.1-leak-fix.patch 
Log Message:
* Thu Oct  2 2008 Hans de Goede <hdegoede at redhat.com> 2.3.1-6
- Fix a small memory leak (which would accumulate over time) thanks to
  Daniel Colascione for the patch (rh 464040)


gkrellm-2.3.1-leak-fix.patch:

--- NEW FILE gkrellm-2.3.1-leak-fix.patch ---
diff -u /tmp/old-clock.c /net/home/danc/rpm/BUILD/gkrellm-2.3.1/src/clock.c
--- /src/clock.c	2008-09-25 20:03:55.000000000 -0400
+++ /net/home/danc/rpm/BUILD/gkrellm-2.3.1/src/clock.c	2008-09-25 20:07:52.000000000 -0400
@@ -396,6 +396,7 @@
 		}
 	gkrellm_draw_decal_markup(pclock, d_clock, clock_string);
 	gkrellm_draw_panel_layers(pclock);
+    g_free(clock_string);
 	}
 
 

Diff finished.  Thu Sep 25 20:08:38 2008


Index: gkrellm.spec
===================================================================
RCS file: /cvs/extras/rpms/gkrellm/devel/gkrellm.spec,v
retrieving revision 1.17
retrieving revision 1.18
diff -u -r1.17 -r1.18
--- gkrellm.spec	10 Sep 2008 08:57:51 -0000	1.17
+++ gkrellm.spec	2 Oct 2008 20:10:23 -0000	1.18
@@ -1,6 +1,6 @@
 Name:           gkrellm
 Version:        2.3.1
-Release:        5%{?dist}
+Release:        6%{?dist}
 Summary:        Multiple stacked system monitors in one process
 Group:          Applications/System
 License:        GPLv3+
@@ -15,6 +15,7 @@
 Patch3:         gkrellm-2.2.7-width.patch
 Patch4:         gkrellm-2.2.9-libdir.patch
 Patch5:         gkrellm-2.3.1-netdb.patch
+Patch6:         gkrellm-2.3.1-leak-fix.patch
 BuildRequires:  gtk2-devel openssl-devel libSM-devel desktop-file-utils gettext
 BuildRequires:  lm_sensors-devel
 BuildRoot:      %{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n)
@@ -64,6 +65,7 @@
 %patch3 -p1 -z .width
 %patch4 -p1 -z .libdir2
 %patch5 -p1 -z .netdb
+%patch6 -p1 -z .leakfix
 cp -a %{SOURCE4} .
 
 for i in gkrellmd.1 gkrellm.1 README Changelog Changelog-plugins.html; do
@@ -173,6 +175,10 @@
 
 
 %changelog
+* Thu Oct  2 2008 Hans de Goede <hdegoede at redhat.com> 2.3.1-6
+- Fix a small memory leak (which would accumulate over time) thanks to
+  Daniel Colascione for the patch (rh 464040)
+
 * Wed Sep 10 2008 Hans de Goede <j.w.r.degoede at hhs.nl> 2.3.1-5
 - Build against openssl instead of gnutls-openssl fixing symbol conflicts when
   using pam_ldap (which uses openssl), this fixes rh 446860