Opened 9 years ago

#1198 new defect

Wrong RTCP loss period calculation with multiple frames per RTP packet

Reported by: bennylp Owned by: nanang
Priority: minor Milestone: Known-Issues-and-Ideas
Component: pjmedia Version: trunk
Keywords: Cc:
Backport to 1.x milestone: Backported:

Description

The calculation in rtcp.c:332:

   period = count * sess->pkt_size * 1000 / sess->clock_rate;
   period *= 1000;

where

  sess->pkt_size = settings->samples_per_frame;

Related to this is the hardcodec G.722 samples_per_frame value in stream.c:2221 :

#if defined(PJMEDIA_HANDLE_G722_MPEG_BUG) && (PJMEDIA_HANDLE_G722_MPEG_BUG!=0)
	/* Special case for G.722 */
	if (info->fmt.pt == PJMEDIA_RTP_PT_G722) {
	    rtcp_setting.clock_rate = 8000;
	    rtcp_setting.samples_per_frame = 160;
	}
#endif

Change History (0)

Note: See TracTickets for help on using tickets.