[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