Re: [kamailio/kamailio] benchmark: properly free previous allocated memory in case of error (d7770ca)
@henningw this is wrong no? it should be ``` pkg_free(bmt->start); shm_free(bmt); ``` instead.
you can't access ``bmt->start`` if you freed ``bmt`` first, no?
Of course, I missed that one in the review, will fix it.
From: Victor Seva notifications@github.com Sent: Mittwoch, 22. März 2023 13:54 To: Kamailio Devel List sr-dev@lists.kamailio.org Cc: Subscribed subscribed@noreply.github.com Subject: [sr-dev] Re: [kamailio/kamailio] benchmark: properly free previous allocated memory in case of error (d7770ca)
@henningwhttps://github.com/henningw this is wrong no? it should be
pkg_free(bmt->start);
shm_free(bmt);
instead.
you can't access bmt->start if you freed bmt first, no?
— Reply to this email directly, view it on GitHubhttps://github.com/kamailio/kamailio/commit/d7770cae2ee68be7184298b2bb41cb0ba3ebb47d#commitcomment-105547932, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ABO7UZJCE4X2DWLVEP24LXLW5LY4XANCNFSM6AAAAAAWDYZ6XY. You are receiving this because you are subscribed to this thread.Message ID: kamailio/kamailio/commit/d7770cae2ee68be7184298b2bb41cb0ba3ebb47d/105547932@github.com
-
Henning Westerholt -
Victor Seva