[Asterisk-code-review] astobj2.c: Add excessive ref count trap. (asterisk[14])

Richard Mudgett asteriskteam at digium.com
Tue Jan 24 14:16:50 CST 2017


Richard Mudgett has uploaded a new change for review. ( https://gerrit.asterisk.org/4802 )

Change subject: astobj2.c: Add excessive ref count trap.
......................................................................

astobj2.c: Add excessive ref count trap.

Change-Id: I32e6a589cf9009450e4ff7cb85c07c9d9ef7fe4a
---
M main/astobj2.c
1 file changed, 14 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/02/4802/1

diff --git a/main/astobj2.c b/main/astobj2.c
index 314bedd..71a81ea 100644
--- a/main/astobj2.c
+++ b/main/astobj2.c
@@ -526,6 +526,20 @@
 
 	if (0 < current_value) {
 		/* The object still lives. */
+#define EXCESSIVE_REF_COUNT		100000
+
+		if (EXCESSIVE_REF_COUNT <= current_value && ret < EXCESSIVE_REF_COUNT) {
+			char excessive_ref_buf[100];
+
+			/* We just reached or went over the excessive ref count trigger */
+			snprintf(excessive_ref_buf, sizeof(excessive_ref_buf),
+				"Excessive refcount %d reached on ao2 object %p",
+				current_value, user_data);
+			ast_log(__LOG_ERROR, file, line, func, "%s\n", excessive_ref_buf);
+
+			__ast_assert_failed(0, excessive_ref_buf, file, line, func);
+		}
+
 		if (ref_log && tag) {
 			fprintf(ref_log, "%p,%s%d,%d,%s,%d,%s,%d,%s\n", user_data,
 				(delta < 0 ? "" : "+"), delta, ast_get_tid(),

-- 
To view, visit https://gerrit.asterisk.org/4802
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I32e6a589cf9009450e4ff7cb85c07c9d9ef7fe4a
Gerrit-PatchSet: 1
Gerrit-Project: asterisk
Gerrit-Branch: 14
Gerrit-Owner: Richard Mudgett <rmudgett at digium.com>



More information about the asterisk-code-review mailing list