Ticket #834 (closed defect: fixed)

Opened 10 years ago

Last modified 10 years ago

Delay buffer's mutex leaks (thanks Girish Kumar for the report).

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

Description (last modified by nanang) (diff)

The original report can be found here.

Identified cases:

  • echo_common.c: pjmedia_echo_destroy() doesn't call pjmedia_delay_buf_destroy().
  • conference bridge: sound/master port destructor doesn't call pjmedia_delay_buf_destroy().
  • internal delaybuf: failure in some steps of delaybuf creation doesn't destroy the mutex.

The corresponding ticket for 1.0.x branch is ticket #835.

Change History

comment:1 Changed 10 years ago by nanang

  • Description modified (diff)

comment:2 Changed 10 years ago by nanang

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

In r2728:

  • Added calls to delay buf destructor in conference.c and echo_common.c.
  • Moved mutex creation to the end of pjmedia_delay_buf_create().
  • Deprecated pjmedia_conf_add_passive_port().
Note: See TracTickets for help on using tickets.