[Pidgin] #12617: Patch to add out-of-band DTMF support

Pidgin trac at pidgin.im
Mon Nov 8 14:32:42 EST 2010


#12617: Patch to add out-of-band DTMF support
-----------------------------------------+----------------------------------
 Reporter:  BigBrownChunx                |        Owner:  darkrain42
     Type:  patch                        |       Status:  new       
Milestone:  Patches Needing Improvement  |    Component:  libpurple 
  Version:  2.7.3                        |   Resolution:            
 Keywords:  dtmf media                   |  
-----------------------------------------+----------------------------------

Comment(by darkrain42):

 BigBrownChunx, another update after banging my head against hiptobecubic's
 backtraces in #pidgin:

 This code crashes when the media session is a !GoogleSession, not
 !JingleSession (i.e. in the expected use case of Google Voice).  Two
 changes needed:
    * Only send XEP-0181 DTMF if the other side advertises support for the
 feature (see the spec)
    * Have a jabber function (the one actually set for the prpl) which
 checks if this is a !JingleSession or !GoogleSession (or something that
 otherwise keeps `jingle_rtp_send_dtmf` from trying to dereference the
 wrong type of variable).

 Type safety in jingle_rtp_send_dtmf
 (`g_return_if_fail(JINGLE_IS_SESSION(...))` probably also good )

-- 
Ticket URL: <http://developer.pidgin.im/ticket/12617#comment:14>
Pidgin <http://pidgin.im>
Pidgin


More information about the Tracker mailing list