[asterisk-commits] dvossel: branch 10 r343900 - /branches/10/channels/chan_sip.c

SVN commits to the Asterisk project asterisk-commits at lists.digium.com
Tue Nov 8 12:29:39 CST 2011


Author: dvossel
Date: Tue Nov  8 12:29:33 2011
New Revision: 343900

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=343900
Log:
Fixes regression caused by r343635

There was a missing unlock for a function return that is only
present in Asterisk 10 and Asterisk Trunk.

(closes issue ASTERISK-18839)
Reported by: Michael L. Young
Patches:
    asterisk-18839-missing-lock-trunk-v2.diff (License #5026) patch uploaded by Michael L. Young


Modified:
    branches/10/channels/chan_sip.c

Modified: branches/10/channels/chan_sip.c
URL: http://svnview.digium.com/svn/asterisk/branches/10/channels/chan_sip.c?view=diff&rev=343900&r1=343899&r2=343900
==============================================================================
--- branches/10/channels/chan_sip.c (original)
+++ branches/10/channels/chan_sip.c Tue Nov  8 12:29:33 2011
@@ -25889,6 +25889,7 @@
 		peer_mailboxes_to_str(&mailbox_str, peer);
 		/* if there is no mailbox do nothing */
 		if (ast_strlen_zero(mailbox_str->str)) {
+			ao2_unlock(peer);
 			return -1;
 		}
 		ao2_unlock(peer);




More information about the asterisk-commits mailing list