[asterisk-commits] tilghman: branch 1.6.2 r257210 - in /branches/1.6.2: ./ channels/chan_sip.c

[SVN commits to the Asterisk project]

Author: tilghman
Date: Tue Apr 13 14:20:58 2010
New Revision: 257210

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=257210
Log:
Merged revisions 257191 via svnmerge from 
https://origsvn.digium.com/svn/asterisk/trunk

........
  r257191 | tilghman | 2010-04-13 14:17:48 -0500 (Tue, 13 Apr 2010) | 10 lines
  
  Also unref the pvt when we delete the provisional keepalive job.
  
  (closes issue #16774)
   Reported by: kowalma
   Patches: 
         20100315__issue16774.diff.txt uploaded by tilghman (license 14)
   Tested by: falves11, jamicque
  
  Review: https://reviewboard.asterisk.org/r/591/
........

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

Propchange: branches/1.6.2/
------------------------------------------------------------------------------
Binary property 'trunk-merged' - no diff available.

Modified: branches/1.6.2/channels/chan_sip.c
URL: http://svnview.digium.com/svn/asterisk/branches/1.6.2/channels/chan_sip.c?view=diff&rev=257210&r1=257209&r2=257210
==============================================================================
--- branches/1.6.2/channels/chan_sip.c (original)
+++ branches/1.6.2/channels/chan_sip.c Tue Apr 13 14:20:58 2010
@@ -6069,7 +6069,7 @@
 				}
 			} else {	/* Incoming call, not up */
 				const char *res;
-				AST_SCHED_DEL(sched, p->provisional_keepalive_sched_id);
+				AST_SCHED_DEL_UNREF(sched, p->provisional_keepalive_sched_id, dialog_unref(p, "when you delete the provisional_keepalive_sched_id, you should dec the refcount for the stored dialog ptr"));
 				if (p->hangupcause && (res = hangup_cause2sip(p->hangupcause)))
 					transmit_response_reliable(p, res, &p->initreq);
 				else