Opened 8 years ago

Closed 8 years ago

#1199 closed enhancement (fixed)

Support G722 with 10ms packet length (thanks Erik Waling for the report)

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

Description

There is G722 implementation that uses 10ms packet length. As currently the pjmedia G722 wrapper expects G722 packet length to be (multiple of) 20ms, the incoming 10ms G722 packets cannot be parsed properly, which leads to:

  • no audio
  • divide-by-zero exception

Change History (2)

comment:1 Changed 8 years ago by nanang

In r3414:

  • fixed possible division-by-zero bug when receiving 10ms G722 packets and the feature of remote RTP clock-rate detection is active (PJMEDIA_HANDLE_G722_MPEG_BUG is set).

comment:2 Changed 8 years ago by nanang

  • Resolution set to fixed
  • Status changed from new to closed

(In [3416]) Fix #1199:

  • Modified G722 frame time to 10ms (was 20ms) and frame per packet to 2 (was 1).
  • Updated the detection mechanism of remote G722 frame-length in the stream to be flexible to any G722 frame length setting (was assumed to be always 20ms).
Note: See TracTickets for help on using tickets.