thanks for the report. It would be great if you could create a pull-request on our github tracker about the suggested change in the module, to make it easier to review and possible merge to the code base.
--
Henning Westerholt -
https://skalatan.de/blog/
Kamailio services -
https://gilawa.com
> -----Original Message-----
> From: whliu--- via sr-users
sr-users@lists.kamailio.org
> Sent: Mittwoch, 21. August 2024 08:48
> To: sr-users@lists.kamailio.org
> Cc: whliu@citictel.com
> Subject: [SR-Users] Suspected memory leak in module ims_registrar_pcscf
>
> Hi,
>
> I just tried to send numbers of SIP register to kamailio (release 5.8.2). It is
> reported that there are some memory leakage in shared memory after the
> expiry of all the registration sessions. After drill down the code. function
> save_pending(...) is called. (in ims_registrar_pcscf/save.c). Inside that
> function, it seems that "sec_verify_params" is not deallocated upon the
> function completes.
>
> Findings:
> - sec_verify_params uses the result from function cscf_get_security_verify(...)
> - Function cscf_get_security_verify(...) returns the result of function
> parse_sec_agree(...)
> - Function parse_sec_agree(...) returns "params" where "shm_malloc(...)" is
> called
>
> It seems that the memory pointed by 'sec_verify_params' is not further
> referenced by others. Should shm_free(...) be called when leaving function
> save_pending(...) in order to free the unused memory of 'sec_verify_params'?
>
> Please advice.
>
> Thank you.
>
> Regards,
> Hong
> __________________________________________________________
> Kamailio - Users Mailing List - Non Commercial Discussions To unsubscribe
> send an email to sr-users-leave@lists.kamailio.org
> Important: keep the mailing list in the recipients, do not reply only to the
> sender!
> Edit mailing list options or unsubscribe: