[asterisk-bugs] [Asterisk 0012679]: [patch] Transcoding breaks jitterbuffering

[Asterisk Bug Tracker]

A NOTE has been added to this issue. 
====================================================================== 
http://bugs.digium.com/view.php?id=12679 
====================================================================== 
Reported By:                damjan
Assigned To:                
====================================================================== 
Project:                    Asterisk
Issue ID:                   12679
Category:                   Channels/General
Reproducibility:            always
Severity:                   major
Priority:                   normal
Status:                     feedback
Asterisk Version:           1.4.18 
SVN Branch (only for SVN checkouts, not tarball releases): N/A 
SVN Revision (number only!):  
Disclaimer on File?:        N/A 
Request Review:              
====================================================================== 
Date Submitted:             2008-05-19 03:46 CDT
Last Modified:              2008-09-09 12:04 CDT
====================================================================== 
Summary:                    [patch] Transcoding breaks jitterbuffering
Description: 
Asterisk doesn't copy or regenerate ts, len or seqno when transcoding from
the original frame to the transcoded frame. The jitterbuffer then sees a
frame with bad timing info and fails to dejitter the frame.

====================================================================== 

---------------------------------------------------------------------- 
 (0092263) jolan (reporter) - 2008-09-09 12:04
 http://bugs.digium.com/view.php?id=12679#c92263 
---------------------------------------------------------------------- 
I think this is the root cause of issue http://bugs.digium.com/view.php?id=12670
however this patch doesn't
fix my issue but this one does:

Index: rtp.c
===================================================================
--- rtp.c	(revision 120859)
+++ rtp.c	(working copy)
@@ -2663,8 +2663,10 @@
     if (rtp->lastts > rtp->lastdigitts)
         rtp->lastdigitts = rtp->lastts;
 
+	/* XXX polycoms can't cope with high timestamps on the initial packet
     if (ast_test_flag(f, AST_FRFLAG_HAS_TIMING_INFO))
         rtp->lastts = f->ts * 8;
+	*/ 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2008-09-09 12:04 jolan          Note Added: 0092263                          
======================================================================