[asterisk-commits] mjordan: branch 1.8 r381975 - /branches/1.8/channels/chan_jingle.c

SVN commits to the Asterisk project asterisk-commits at lists.digium.com
Sun Feb 24 16:59:15 CST 2013


Author: mjordan
Date: Sun Feb 24 16:59:11 2013
New Revision: 381975

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=381975
Log:
Set the sin_family on the bind address socket during initialization

Somehow, chan_jingle has managed to operate for years without setting the
sin_family on its bindaddr socket. This patch properly sets the field during
initial module load to AF_INET.

Note that the patch on the issue was modified slightly to change the
initialization of the socket from allocation of a chan_jingle private to the
module initialization, as the bindaddr object (which is static) only needs to
have the address set once.

(closes issue ASTERISK-19341)
Reported by: andre valentin
patches:
  0105-chan_jingle.patch uploaded by avalentin (License 6064)

Modified:
    branches/1.8/channels/chan_jingle.c

Modified: branches/1.8/channels/chan_jingle.c
URL: http://svnview.digium.com/svn/asterisk/branches/1.8/channels/chan_jingle.c?view=diff&rev=381975&r1=381974&r2=381975
==============================================================================
--- branches/1.8/channels/chan_jingle.c (original)
+++ branches/1.8/channels/chan_jingle.c Sun Feb 24 16:59:11 2013
@@ -1926,6 +1926,7 @@
 		ast_log(LOG_WARNING, "Unable to create I/O context\n");
 	}
 
+	bindaddr.sin_family = AF_INET;
 	ast_sockaddr_from_sin(&bindaddr_tmp, &bindaddr);
 	if (ast_find_ourip(&ourip_tmp, &bindaddr_tmp, AF_INET)) {
 		ast_log(LOG_WARNING, "Unable to get own IP address, Jingle disabled\n");




More information about the asterisk-commits mailing list