[Home]

Summary:ASTERISK-16595: [regression][patch] RTP Packets Not Set With QOS
Reporter:Michael L. Young (elguero)Labels:
Date Opened:2010-08-19 23:56:36Date Closed:2010-08-24 13:58:45
Priority:MinorRegression?No
Status:Closed/CompleteComponents:Resources/General
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) qos_18.diff
Description:It would appear that with the merge of the rtp_engine api, the tos is no longer being set on RTP packets through res_rtp_asterisk.  Also, chan_sip is not setting the cos on RTP packets.

The attached patch restores setting the tos on rtp packets.

The patch also sets cos on rtp packets using chan_sip.

****** ADDITIONAL INFORMATION ******

There doesn't seem to be a category for res/res_rtp_asterisk.
Comments:By: Leif Madsen (lmadsen) 2010-08-24 13:21:00

Russell reviewed this, so I'll just commit it.
By: Digium Subversion (svnbot) 2010-08-24 13:56:28

Repository: asterisk
Revision: 283457

U   branches/1.8/channels/chan_sip.c
U   branches/1.8/res/res_rtp_asterisk.c

------------------------------------------------------------------------
r283457 | lmadsen | 2010-08-24 13:56:27 -0500 (Tue, 24 Aug 2010) | 9 lines

Fix issue where TOS is no longer set on RTP packets.
Fix issue where the tos is no longer being set on RTP packets through res_rtp_asterisk.

(closes issue ASTERISK-16595)
Reported by: elguero
Patches:
     qos_18.diff uploaded by elguero (license 37)

Review: https://reviewboard.asterisk.org/r/868
------------------------------------------------------------------------

http://svn.digium.com/view/asterisk?view=rev&revision=283457
By: Digium Subversion (svnbot) 2010-08-24 13:58:44

Repository: asterisk
Revision: 283458

_U  trunk/
U   trunk/channels/chan_sip.c
U   trunk/res/res_rtp_asterisk.c

------------------------------------------------------------------------
r283458 | lmadsen | 2010-08-24 13:58:44 -0500 (Tue, 24 Aug 2010) | 17 lines

Merged revisions 283457 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.8

........
 r283457 | lmadsen | 2010-08-24 13:56:29 -0500 (Tue, 24 Aug 2010) | 9 lines
 
 Fix issue where TOS is no longer set on RTP packets.
 Fix issue where the tos is no longer being set on RTP packets through res_rtp_asterisk.
 
 (closes issue ASTERISK-16595)
 Reported by: elguero
 Patches:
       qos_18.diff uploaded by elguero (license 37)
 
 Review: https://reviewboard.asterisk.org/r/868
........

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

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