ports/165003: [patch] icecast-2.3.2 may leak memory and take extra CPU time

[Eugene Grosbein]

>Number:         165003
>Category:       ports
>Synopsis:       [patch] icecast-2.3.2 may leak memory and take extra CPU time
>Confidential:   no
>Severity:       non-critical
>Priority:       low
>Responsible:    freebsd-ports-bugs
>State:          open
>Quarter:        
>Keywords:       
>Date-Required:
>Class:          sw-bug
>Submitter-Id:   current-users
>Arrival-Date:   Sat Feb 11 20:40:08 UTC 2012
>Closed-Date:
>Last-Modified:
>Originator:     Eugene Grosbein
>Release:        FreeBSD 8.2-STABLE amd64
>Organization:
RDTC JSC
>Environment:
System: FreeBSD grosbein.pp.ru 8.2-STABLE FreeBSD 8.2-STABLE #14: Mon Jan 16 21:33:53 NOVT 2012 root at grosbein.pp.ru:/usr/local/obj/usr/local/src/sys/DADV amd64

>Description:

	icecast2-2.3.2_7,1 built from ports tree may leak memory.
	When it leaks, its size grows and CPU time grows enormously too.
	After it reaches 2Gb for i386 system, it is killed with a signal.

>How-To-Repeat:

	Run busy icecast2 server.

>Fix:

	The following 2 years old patch was taken from
http://old.nabble.com/Memory-leak-on-Icecast-2.3.2---Debian---td26691536.html

	It eliminates the problem fixing memory leak.

--- src/stats.c.orig	2012-02-11 22:28:24.000000000 +0400
+++ src/stats.c	2012-02-11 22:29:17.000000000 +0400
@@ -973,7 +973,11 @@
                 if (event->source)
                 {
                     if (show_mount && strcmp (event->source, show_mount) != 0)
+            	      {
+            	        xmlFree (value);
+            	        xmlFree (name);
                         break;
+                      }
                     srcnode = _find_xml_node(event->source, &src_nodes, node);
                 }
                 else
>Release-Note:
>Audit-Trail:
>Unformatted: