[asterisk-commits] russell: branch 1.8 r317427 - /branches/1.8/apps/app_rpt.c

SVN commits to the Asterisk project asterisk-commits at lists.digium.com
Thu May 5 16:58:50 CDT 2011


Author: russell
Date: Thu May  5 16:58:45 2011
New Revision: 317427

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=317427
Log:
Fix potential memory leak, and use of uninitialized memory.

(closes issue #16476)
Reported by: junky
Patches:
      M16476.diff uploaded by junky (license 177)

Modified:
    branches/1.8/apps/app_rpt.c

Modified: branches/1.8/apps/app_rpt.c
URL: http://svnview.digium.com/svn/asterisk/branches/1.8/apps/app_rpt.c?view=diff&rev=317427&r1=317426&r2=317427
==============================================================================
--- branches/1.8/apps/app_rpt.c (original)
+++ branches/1.8/apps/app_rpt.c Thu May  5 16:58:45 2011
@@ -1963,7 +1963,11 @@
 		return;
 	}
 	n = finddelim(astr,astrs,100);
-	if (n < 1) return;
+	if (n < 1) {
+		ast_free(str);
+		ast_free(astr);
+		return;
+	}
 	ast_mutex_lock(&myrpt->statpost_lock);
 	seq = ++myrpt->statpost_seqno;
 	ast_mutex_unlock(&myrpt->statpost_lock);
@@ -12188,7 +12192,7 @@
 						// ctcss code autopatch initiate
 						if (strstr((char *)f->data.ptr,"/M/")&& !myrpt->macropatch)
 						{
-						    char value[16];
+							char value[16] = "";
 							strcat(value,"*6");
 							myrpt->macropatch=1;
 							rpt_mutex_lock(&myrpt->lock);




More information about the asterisk-commits mailing list