[OpenSIPS-Devel] [OpenSIPS/opensips] f7defa: rtp_relay: fix invalid contexts leaked in list

Răzvan Crainea noreply at github.com
Tue Sep 5 09:20:19 UTC 2023


  Branch: refs/heads/3.3
  Home:   https://github.com/OpenSIPS/opensips
  Commit: f7defa0161b70794b39331c325afc18519f84f8c
      https://github.com/OpenSIPS/opensips/commit/f7defa0161b70794b39331c325afc18519f84f8c
  Author: Razvan Crainea <razvan at opensips.org>
  Date:   2023-09-04 (Mon, 04 Sep 2023)

  Changed paths:
    M modules/rtp_relay/rtp_relay_ctx.c

  Log Message:
  -----------
  rtp_relay: fix invalid contexts leaked in list

When a session would have been established on a provisional reply with
body (i.e. 183), the rtp contexts would have been stored in the list.
However, if the call would not establish (with a 200 OK), there would be
no one removing it from the list, even though the context was released -
this resulted in unexpected and almost impossible to track down invalid
memory usage.
The current patch makes sure that the context is removed from the list
if the call does not establish.

Many thanks to Rob Moore, Daniel Bryars and Steven Ayre from Dubber for
pushing an enormous amount of time into troubleshooting and debugging
this, as well as to Liviu Chircu for guiding the troubleshooting!

(cherry picked from commit d6ef279f63aa0682bf0fb7ef1d30a6abdfb8b942)





More information about the Devel mailing list