[asterisk-bugs] [Asterisk 0017890]: [regression][patch] RTP Packets Not Set With QOS

[Asterisk Bug Tracker]

A NOTE has been added to this issue. 
====================================================================== 
https://issues.asterisk.org/view.php?id=17890 
====================================================================== 
Reported By:                elguero
Assigned To:                lmadsen
====================================================================== 
Project:                    Asterisk
Issue ID:                   17890
Category:                   Resources/General
Reproducibility:            always
Severity:                   minor
Priority:                   normal
Status:                     closed
Target Version:             1.8.0
Asterisk Version:           SVN 
JIRA:                        
Regression:                 No 
Reviewboard Link:           https://reviewboard.asterisk.org/r/868 
SVN Branch (only for SVN checkouts, not tarball releases): 1.8 
SVN Revision (number only!): 282960 
Request Review:              
Resolution:                 fixed
Fixed in Version:           
====================================================================== 
Date Submitted:             2010-08-19 23:56 CDT
Last Modified:              2010-08-24 13:58 CDT
====================================================================== 
Summary:                    [regression][patch] RTP Packets Not Set With QOS
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.
====================================================================== 

---------------------------------------------------------------------- 
 (0126283) svnbot (reporter) - 2010-08-24 13:58
 https://issues.asterisk.org/view.php?id=17890#c126283 
---------------------------------------------------------------------- 
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 https://issues.asterisk.org/view.php?id=17890)
  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 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2010-08-24 13:58 svnbot         Checkin                                      
2010-08-24 13:58 svnbot         Note Added: 0126283                          
======================================================================