[asterisk-commits] phsultan: trunk r139281 - /trunk/channels/chan_gtalk.c

[SVN commits to the Asterisk project]

Author: phsultan
Date: Thu Aug 21 04:55:31 2008
New Revision: 139281

URL: http://svn.digium.com/view/asterisk?view=rev&rev=139281
Log:
Fix two memory leaks in chan_gtalk, thanks Eliel!
(closes issue #13310)
Reported by: eliel
Patches:
      chan_gtalk.c.patch uploaded by eliel (license 64)

Modified:
    trunk/channels/chan_gtalk.c

Modified: trunk/channels/chan_gtalk.c
URL: http://svn.digium.com/view/asterisk/trunk/channels/chan_gtalk.c?view=diff&rev=139281&r1=139280&r2=139281
==============================================================================
--- trunk/channels/chan_gtalk.c (original)
+++ trunk/channels/chan_gtalk.c Thu Aug 21 04:55:31 2008
@@ -1619,6 +1619,7 @@
 		client->connection = ast_aji_get_client(sender);
 		if (!client->connection) {
 			ast_log(LOG_ERROR, "No XMPP client to talk to, us (partial JID) : %s\n", sender);
+			ASTOBJ_UNREF(client, gtalk_member_destroy);
 			return NULL;
 		}
 	}
@@ -1960,6 +1961,7 @@
 						ASTOBJ_UNLOCK(iterator);
 					});
 					ASTOBJ_CONTAINER_LINK(&gtalk_list, member);
+					ASTOBJ_UNREF(member, gtalk_member_destroy);
 				} else {
 					ASTOBJ_UNLOCK(member);
 					ASTOBJ_UNREF(member, gtalk_member_destroy);