Opened 12 years ago

Closed 12 years ago

#478 closed defect (fixed)

Handle duplicated/misordered incoming DTMF packets

Reported by: nanang Owned by: nanang
Priority: normal Milestone: release-0.9.0
Component: pjmedia Version: trunk
Keywords: Cc:
Backport to 1.x milestone: Backported:

Description (last modified by bennylp)

The incoming RFC 2833 DTMF handler does not handle the case where RTP packets arrive out of order. In this case, the packets may cause stream.c to detect them as multiple DTMF events.

Attachments (1)

dtmf.patch (3.1 KB) - added by nanang 12 years ago.

Download all attachments as: .zip

Change History (3)

Changed 12 years ago by nanang

comment:1 Changed 12 years ago by nanang

Attachment dtmf.patch:

  • Added new API pjmedia_rtp_session_update2() which behaves like pjmedia_rtp_session_update() with additional option to validate payload type
  • Changed in stream.c:
    • replaced pjmedia_rtp_session_update() with pjmedia_rtp_session_update2(), so the RTP session validation will avoid payload type checking when received PT == DTMF event PT
    • moved handle_incoming_dtmf() from before to after pjmedia_rtp_session_update2()

comment:2 Changed 12 years ago by bennylp

  • Description modified (diff)
  • Resolution set to fixed
  • Status changed from new to closed

Committed in r1798 with some modification, thanks.

Note: See TracTickets for help on using tickets.