[OpenSIPS-Devel] [OpenSIPS/opensips] ae413f: rtp_relay: fix invalid contexts leaked in list
Branch: refs/heads/3.2 Home: https://github.com/OpenSIPS/opensips Commit: ae413ff72584e63e21556dff1d3f60568e5256f8 https://github.com/OpenSIPS/opensips/commit/ae413ff72584e63e21556dff1d3f60568e5256f8 Author: Razvan Crainea Date: 2023-09-05 (Tue, 05 Sep 2023) Changed paths: M lib/list.h 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) ___ Devel mailing list Devel@lists.opensips.org http://lists.opensips.org/cgi-bin/mailman/listinfo/devel
[OpenSIPS-Devel] [OpenSIPS/opensips] f7defa: rtp_relay: fix invalid contexts leaked in list
Branch: refs/heads/3.3 Home: https://github.com/OpenSIPS/opensips Commit: f7defa0161b70794b39331c325afc18519f84f8c https://github.com/OpenSIPS/opensips/commit/f7defa0161b70794b39331c325afc18519f84f8c Author: Razvan Crainea 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) ___ Devel mailing list Devel@lists.opensips.org http://lists.opensips.org/cgi-bin/mailman/listinfo/devel
[OpenSIPS-Devel] [OpenSIPS/opensips] 6aae49: rtp_relay: fix invalid contexts leaked in list
Branch: refs/heads/3.4 Home: https://github.com/OpenSIPS/opensips Commit: 6aae498265fb6723881acba1b6db2ce3a137d7ca https://github.com/OpenSIPS/opensips/commit/6aae498265fb6723881acba1b6db2ce3a137d7ca Author: Razvan Crainea 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) ___ Devel mailing list Devel@lists.opensips.org http://lists.opensips.org/cgi-bin/mailman/listinfo/devel
[OpenSIPS-Devel] [OpenSIPS/opensips] d6ef27: rtp_relay: fix invalid contexts leaked in list
Branch: refs/heads/master Home: https://github.com/OpenSIPS/opensips Commit: d6ef279f63aa0682bf0fb7ef1d30a6abdfb8b942 https://github.com/OpenSIPS/opensips/commit/d6ef279f63aa0682bf0fb7ef1d30a6abdfb8b942 Author: Razvan Crainea 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! ___ Devel mailing list Devel@lists.opensips.org http://lists.opensips.org/cgi-bin/mailman/listinfo/devel
