Hello, 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.
Thanks, Henning -- Henning Westerholt - https://skalatan.de/blog/ Kamailio services - https://gilawa.com > -----Original Message----- > From: whliu--- via sr-users <[email protected]> > Sent: Mittwoch, 21. August 2024 08:48 > To: [email protected] > Cc: [email protected] > 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 [email protected] > Important: keep the mailing list in the recipients, do not reply only to the > sender! > Edit mailing list options or unsubscribe: __________________________________________________________ Kamailio - Users Mailing List - Non Commercial Discussions To unsubscribe send an email to [email protected] Important: keep the mailing list in the recipients, do not reply only to the sender! Edit mailing list options or unsubscribe:
