Opened 11 years ago

Closed 10 years ago

#834 closed defect (fixed)

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)

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 (2)

comment:1 Changed 11 years ago by nanang

  • Description modified (diff)

comment:2 Changed 10 years ago by nanang

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

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.