[asterisk-bugs] [Asterisk 0018397]: IAX2 CODEC_PRES wrong (offset error?)

[Asterisk Bug Tracker]

A NOTE has been added to this issue. 
====================================================================== 
https://issues.asterisk.org/view.php?id=18397 
====================================================================== 
Reported By:                birgita
Assigned To:                tilghman
====================================================================== 
Project:                    Asterisk
Issue ID:                   18397
Category:                   Channels/chan_iax2
Reproducibility:            always
Severity:                   minor
Priority:                   normal
Status:                     closed
Asterisk Version:           SVN 
JIRA:                        
Regression:                 No 
Reviewboard Link:            
SVN Branch (only for SVN checkouts, not tarball releases): N/A 
SVN Revision (number only!): 296230 
Request Review:              
Resolution:                 fixed
Fixed in Version:           
====================================================================== 
Date Submitted:             2010-11-29 08:41 CST
Last Modified:              2010-12-01 11:01 CST
====================================================================== 
Summary:                    IAX2 CODEC_PRES wrong (offset error?)
Description: 
On an incoming IAX2 call, the console reports (correctly):

Tx-Frame Retry[000] -- OSeqno: 000 ISeqno: 000 Type: IAX     Subclass: NEW
   
   Timestamp: 00005ms  SCall: 09646  DCall: 00000 [94.12.29.127:2705]
   VERSION         : 2
   CODEC_PREFS     : (gsm|ulaw)
   FORMAT          : 2
   FORMAT2         : gsm
   CAPABILITY      : 6
   CAPABILITY2     : unknown
   ADSICPE         : 2

However, Asterisk sends over the wire as value for CODEC_PREFS: "CD"
This is incorrect. It should be "BC":

A = "AUDIO_G723",
B = "AUDIO_GSM",
C = "AUDIO_ULAW",
D = "AUDIO_ALAW",

In this case codec negotiation does not fail because of FORMAT(2).
====================================================================== 

---------------------------------------------------------------------- 
 (0129240) svnbot (reporter) - 2010-12-01 11:01
 https://issues.asterisk.org/view.php?id=18397#c129240 
---------------------------------------------------------------------- 
Repository: asterisk
Revision: 296992

_U  branches/1.8/
U   branches/1.8/include/asterisk/frame.h

------------------------------------------------------------------------
r296992 | tilghman | 2010-12-01 11:01:57 -0600 (Wed, 01 Dec 2010) | 19
lines

Merged revisions 296991 via svnmerge from 
https://origsvn.digium.com/svn/asterisk/branches/1.6.2

................
  r296991 | tilghman | 2010-12-01 11:01:00 -0600 (Wed, 01 Dec 2010) | 12
lines
  
  Merged revisions 296990 via svnmerge from 
  https://origsvn.digium.com/svn/asterisk/branches/1.4
  
  ........
    r296990 | tilghman | 2010-12-01 10:59:26 -0600 (Wed, 01 Dec 2010) | 5
lines
    
    Clarify documentation on how we store codec preference lists.
    
    (closes issue https://issues.asterisk.org/view.php?id=18397)
     Reported by: birgita
  ........
................

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

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

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2010-12-01 11:01 svnbot         Checkin                                      
2010-12-01 11:01 svnbot         Note Added: 0129240                          
======================================================================