[asterisk-commits] wdoekes: branch 11 r424177 - in /branches/11: ./ channels/chan_sip.c

SVN commits to the Asterisk project asterisk-commits at lists.digium.com
Wed Oct 1 04:52:18 CDT 2014


Author: wdoekes
Date: Wed Oct  1 04:52:13 2014
New Revision: 424177

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=424177
Log:
chan_sip: Remove excess ref of realtime peer before sip_poke_peer.

The peer is referenced at the end of sip_poke_peer, it should not get
an extra ref before the call to sip_poke_peer. This fixes a memory
leak.

ASTERISK-22945 #close
Reported by: ibercom
Tested by: Yuriy Gorlichenko
Patches:
  asterisk11.patch uploaded by ibercom (License #6599)

Review: https://reviewboard.asterisk.org/r/4031/
........

Merged revisions 424176 from http://svn.asterisk.org/svn/asterisk/branches/1.8

Modified:
    branches/11/   (props changed)
    branches/11/channels/chan_sip.c

Propchange: branches/11/
------------------------------------------------------------------------------
Binary property 'branch-1.8-merged' - no diff available.

Modified: branches/11/channels/chan_sip.c
URL: http://svnview.digium.com/svn/asterisk/branches/11/channels/chan_sip.c?view=diff&rev=424177&r1=424176&r2=424177
==============================================================================
--- branches/11/channels/chan_sip.c (original)
+++ branches/11/channels/chan_sip.c Wed Oct  1 04:52:13 2014
@@ -31209,7 +31209,6 @@
 
 		/* Startup regular pokes */
 		if (!devstate_only && enablepoke) {
-			sip_ref_peer(peer, "schedule qualify");
 			sip_poke_peer(peer, 0);
 		}
 	}




More information about the asterisk-commits mailing list