[Home]

Summary:ASTERISK-02102: [patch] debug peer port correction
Reporter:cherso (cherso)Labels:
Date Opened:2004-07-25 12:26:50Date Closed:2008-01-15 15:04:03.000-0600
Priority:MinorRegression?No
Status:Closed/CompleteComponents:Core/General
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) rtp_port_debug.diff
Description:debug message always shows port ip:0 when using only audio

/* Setup audio port number */
sin.sin_port = htons(portno);
if (p->rtp && sin.sin_port)
ast_rtp_set_peer(p->rtp, &sin);
if (sipdebug)
ast_verbose("Peer audio RTP is at port %s:%d\n", ast_inet_ntoa(iabuf, sizeof(iabuf), sin.sin_addr), ntohs(sin.sin_port));
/* Setup video port number */
sin.sin_port = htons(vportno);
if (p->vrtp && sin.sin_port)
ast_rtp_set_peer(p->vrtp, &sin);
if (sipdebug)
ast_verbose("Peer video RTP is at port %s:%d\n", ast_inet_ntoa(iabuf, sizeof(iabuf), sin.sin_addr), ntohs(sin.sin_port));
Comments:By: cherso (cherso) 2004-07-25 12:31:17

process_sdp function

/* Setup audio port number */
sin.sin_port = htons(portno);
if (p->rtp && sin.sin_port) {
ast_rtp_set_peer(p->rtp, &sin);
if (sipdebug)
ast_verbose("Peer audio RTP is at port %s:%d\n", ast_inet_ntoa(iabuf, sizeof(iabuf), sin.sin_addr), ntohs(sin.sin_port));
ast_log(LOG_DEBUG,"Peer audio RTP is at port %s:%d\n", ast_inet_ntoa(iabuf, sizeof(iabuf), sin.sin_addr), ntohs(sin.sin_port));
}
/* Setup video port number */
sin.sin_port = htons(vportno);
if (p->vrtp && sin.sin_port) {
ast_rtp_set_peer(p->vrtp, &sin);
if (sipdebug)
ast_verbose("Peer video RTP is at port %s:%d\n", ast_inet_ntoa(iabuf, sizeof(iabuf), sin.sin_addr), ntohs(sin.sin_port));
     ast_log(LOG_DEBUG,"Peer video RTP is at port %s:%d\n", ast_inet_ntoa(iabuf, sizeof(iabuf), sin.sin_addr), ntohs(sin.sin_port));
}
By: Brian West (bkw918) 2004-07-25 13:00:13

attach a diff -u please.
By: twisted (twisted) 2004-07-30 12:28:17

just a friendly reminder, attach a diff -u...
By: cherso (cherso) 2004-07-30 13:40:32

ok done
By: Mark Spencer (markster) 2004-07-30 22:45:45

Fixed in CVS, but I modified your patch in ways that seemed to make sense, but it should still do what you want.  Thanks!
By: Digium Subversion (svnbot) 2008-01-15 15:04:03.000-0600

Repository: asterisk
Revision: 3548

U   trunk/channels/chan_sip.c

------------------------------------------------------------------------
r3548 | markster | 2008-01-15 15:04:03 -0600 (Tue, 15 Jan 2008) | 2 lines

Improve debugging of RTP ports (bug ASTERISK-2102, heavily modified)

------------------------------------------------------------------------

http://svn.digium.com/view/asterisk?view=rev&revision=3548