[asterisk-commits] twilson: trunk r290543 - in /trunk: ./ res/res_rtp_asterisk.c

[SVN commits to the Asterisk project]

Author: twilson
Date: Tue Oct  5 23:47:57 2010
New Revision: 290543

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=290543
Log:
Merged revisions 290542 via svnmerge from 
https://origsvn.digium.com/svn/asterisk/branches/1.8

........
  r290542 | twilson | 2010-10-05 21:35:51 -0700 (Tue, 05 Oct 2010) | 6 lines
  
  Don't try to send RTP when remote_address is null
  
  It is possible for ast_rtp_stop() to be called which will clear the remote
  address and cause the sendto to fail and spam warnings. Don't send in this
  case.
........

Modified:
    trunk/   (props changed)
    trunk/res/res_rtp_asterisk.c

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

Modified: trunk/res/res_rtp_asterisk.c
URL: http://svnview.digium.com/svn/asterisk/trunk/res/res_rtp_asterisk.c?view=diff&rev=290543&r1=290542&r2=290543
==============================================================================
--- trunk/res/res_rtp_asterisk.c (original)
+++ trunk/res/res_rtp_asterisk.c Tue Oct  5 23:47:57 2010
@@ -1945,6 +1945,11 @@
 
 	ast_rtp_instance_get_remote_address(instance1, &remote_address);
 
+	if (ast_sockaddr_isnull(&remote_address)) {
+		ast_debug(1, "Remote address is null, most likely RTP has been stopped\n");
+		return 0;
+	}
+
 	/* Send the packet back out */
 	res = rtp_sendto(instance1, (void *)rtpheader, len, 0, &remote_address);
 	if (res < 0) {