[asterisk-bugs] [Asterisk 0013013]: [patch] unwrap_timestamp() is bogus for video frames

[noreply at bugs.digium.com]

The following issue has been UPDATED. 
====================================================================== 
http://bugs.digium.com/view.php?id=13013 
====================================================================== 
Reported By:                jpgrayson
Assigned To:                
====================================================================== 
Project:                    Asterisk
Issue ID:                   13013
Category:                   Channels/chan_iax2
Reproducibility:            always
Severity:                   major
Priority:                   normal
Status:                     new
Asterisk Version:           SVN 
SVN Branch (only for SVN checkouts, not tarball releases):  1.4  
SVN Revision (number only!):  
Disclaimer on File?:        N/A 
Request Review:              
====================================================================== 
Date Submitted:             07-07-2008 14:19 CDT
Last Modified:              07-07-2008 14:21 CDT
====================================================================== 
Summary:                    [patch] unwrap_timestamp() is bogus for video frames
Description: 
In chan_iax2, unwrap_timestamp() takes the 16-bit timestamp found in the
iax2 packet header and the 32-bit timestamp in the chan_iax2_pvt structure
and reconstructs a full 32-bit timestamp for the incoming packet.

The problem is that this function is totally bogus for video frames. The
fundamental problem is that video mini frames only have 15-bits of
timestamp in the packet header, not 16-bits like voice frames.

The consequence of this is that asterisk will periodically fail to detect
timestamp rollover for video frames. This results in big swaths of video
packets to be thrown in the bit bucket.

unwrap_timestamp() needs to account for video frames which only have
15-bits of timestamp information.
====================================================================== 

Issue History 
Date Modified   Username       Field                    Change               
====================================================================== 
07-07-08 14:21  jpgrayson      Summary                  unwrap_timestamp() is
bogus for video frames => [patch] unwrap_timestamp() is bogus for video frames
======================================================================