[asterisk-commits] rmudgett: branch 12 r399583 - /branches/12/apps/app_queue.c

[SVN commits to the Asterisk project]

Author: rmudgett
Date: Fri Sep 20 19:22:30 2013
New Revision: 399583

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=399583
Log:
app_queue: Fix json blob ref leak.

The json ref from queue_member_blob_create() was never released.

Modified:
    branches/12/apps/app_queue.c

Modified: branches/12/apps/app_queue.c
URL: http://svnview.digium.com/svn/asterisk/branches/12/apps/app_queue.c?view=diff&rev=399583&r1=399582&r2=399583
==============================================================================
--- branches/12/apps/app_queue.c (original)
+++ branches/12/apps/app_queue.c Fri Sep 20 19:22:30 2013
@@ -1952,12 +1952,29 @@
 			agent_snapshot, type, blob);
 }
 
+/*!
+ * \internal
+ * \brief Publish the member blob.
+ * \since 12.0.0
+ *
+ * \param type Stasis message type to publish.
+ * \param blob The information being published.
+ *
+ * \note The json blob reference is passed to this function.
+ *
+ * \return Nothing
+ */
 static void queue_publish_member_blob(struct stasis_message_type *type, struct ast_json *blob)
 {
 	RAII_VAR(struct ast_json_payload *, payload, NULL, ao2_cleanup);
 	RAII_VAR(struct stasis_message *, msg, NULL, ao2_cleanup);
 
+	if (!blob) {
+		return;
+	}
+
 	payload = ast_json_payload_create(blob);
+	ast_json_unref(blob);
 	if (!payload) {
 		return;
 	}
@@ -6729,7 +6746,8 @@
 	return res;
 }
 
-static int publish_queue_member_pause(struct call_queue *q, struct member *member, const char *reason) {
+static int publish_queue_member_pause(struct call_queue *q, struct member *member, const char *reason)
+{
 	struct ast_json *json_blob = queue_member_blob_create(q, member);
 
 	if (!json_blob) {