[asterisk-commits] bebuild: tag 1.8.32.0-rc2 r427548 - in /tags/1.8.32.0-rc2: ./ include/asteris...

SVN commits to the Asterisk project asterisk-commits at lists.digium.com
Fri Nov 7 14:16:13 CST 2014


Author: bebuild
Date: Fri Nov  7 14:16:06 2014
New Revision: 427548

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=427548
Log:
Merge 427380 for rc2; update ChangeLog

Modified:
    tags/1.8.32.0-rc2/   (props changed)
    tags/1.8.32.0-rc2/ChangeLog
    tags/1.8.32.0-rc2/include/asterisk/stringfields.h
    tags/1.8.32.0-rc2/main/utils.c

Propchange: tags/1.8.32.0-rc2/
------------------------------------------------------------------------------
--- svn:mergeinfo (original)
+++ svn:mergeinfo Fri Nov  7 14:16:06 2014
@@ -1,1 +1,2 @@
+/branches/1.8:427380
 /trunk:394552,394567

Modified: tags/1.8.32.0-rc2/ChangeLog
URL: http://svnview.digium.com/svn/asterisk/tags/1.8.32.0-rc2/ChangeLog?view=diff&rev=427548&r1=427547&r2=427548
==============================================================================
--- tags/1.8.32.0-rc2/ChangeLog (original)
+++ tags/1.8.32.0-rc2/ChangeLog Fri Nov  7 14:16:06 2014
@@ -1,3 +1,21 @@
+2014-11-07  Asterisk Development Team <asteriskteam at digium.com>
+
+	* Asterisk 1.8.32.0-rc2 Released.
+
+2014-11-06 09:05 +0000 [r427380]  Corey Farrell <git at cfware.com>
+
+	* Fix unintential memory retention in stringfields.
+
+	  * Fix missing / unreachable calls to
+	    __ast_string_field_release_active.
+	  * Reset pool->used to zero when the current pool->active reaches
+	    zero.
+
+	ASTERISK-24307 #close
+	Reported by: Etienne Lessard
+	Tested by: ibercom, Etienne Lessard
+	Review: https://reviewboard.asterisk.org/r/4114/
+
 2014-11-03  Asterisk Development Team <asteriskteam at digium.com>
 
 	* Asterisk 1.8.32.0-rc1 Released.

Modified: tags/1.8.32.0-rc2/include/asterisk/stringfields.h
URL: http://svnview.digium.com/svn/asterisk/tags/1.8.32.0-rc2/include/asterisk/stringfields.h?view=diff&rev=427548&r1=427547&r2=427548
==============================================================================
--- tags/1.8.32.0-rc2/include/asterisk/stringfields.h (original)
+++ tags/1.8.32.0-rc2/include/asterisk/stringfields.h Fri Nov  7 14:16:06 2014
@@ -319,14 +319,16 @@
 	const char *__d__ = (data);											\
 	size_t __dlen__ = (__d__) ? strlen(__d__) + 1 : 1;								\
 	ast_string_field *__p__ = (ast_string_field *) (ptr);								\
+	ast_string_field target = *__p__;										\
 	if (__dlen__ == 1) {												\
 		__ast_string_field_release_active((x)->__field_mgr_pool, *__p__);					\
 		*__p__ = __ast_string_field_empty;									\
 	} else if ((__dlen__ <= AST_STRING_FIELD_ALLOCATION(*__p__)) ||							\
 		   (!__ast_string_field_ptr_grow(&(x)->__field_mgr, &(x)->__field_mgr_pool, __dlen__, __p__)) ||	\
-		   (*__p__ = __ast_string_field_alloc_space(&(x)->__field_mgr, &(x)->__field_mgr_pool, __dlen__))) {	\
-		if (*__p__ != (*ptr)) {											\
-			__ast_string_field_release_active((x)->__field_mgr_pool, (*ptr));				\
+		   (target = __ast_string_field_alloc_space(&(x)->__field_mgr, &(x)->__field_mgr_pool, __dlen__))) {	\
+		if (target != *__p__) {											\
+			__ast_string_field_release_active((x)->__field_mgr_pool, *__p__);				\
+			*__p__ = target;										\
 		}													\
 		memcpy(* (void **) __p__, __d__, __dlen__);								\
 	}														\

Modified: tags/1.8.32.0-rc2/main/utils.c
URL: http://svnview.digium.com/svn/asterisk/tags/1.8.32.0-rc2/main/utils.c?view=diff&rev=427548&r1=427547&r2=427548
==============================================================================
--- tags/1.8.32.0-rc2/main/utils.c (original)
+++ tags/1.8.32.0-rc2/main/utils.c Fri Nov  7 14:16:06 2014
@@ -1919,9 +1919,13 @@
 	for (pool = pool_head, prev = NULL; pool; prev = pool, pool = pool->prev) {
 		if ((ptr >= pool->base) && (ptr <= (pool->base + pool->size))) {
 			pool->active -= AST_STRING_FIELD_ALLOCATION(ptr);
-			if ((pool->active == 0) && prev) {
-				prev->prev = pool->prev;
-				ast_free(pool);
+			if (pool->active == 0) {
+				if (prev) {
+					prev->prev = pool->prev;
+					ast_free(pool);
+				} else {
+					pool->used = 0;
+				}
 			}
 			break;
 		}
@@ -1964,6 +1968,11 @@
 	res = vsnprintf(target, available, format, ap1);
 	if (res < 0) {
 		/* Are we out of memory? */
+		return;
+	}
+	if (res == 0) {
+		__ast_string_field_release_active(*pool_head, *ptr);
+		*ptr = __ast_string_field_empty;
 		return;
 	}
 	needed = (size_t)res + 1; /* NUL byte */




More information about the asterisk-commits mailing list