Fix a crash on applications shared memory allocation when Diameter CEA contains Auth_Applications or Acct_Applications AVPs not "embedded" into Vendor_Specific grouped AVP Before only the memory for applications found on Vendor_Specific AVP was reserved, but if Acct_Applications or Auth_Applications not Vendor Specific is found, you need to reserve memory for the number of Acct_Applications or Auth_Applications AVPs found multiplied by the number of Supported Vendor ID AVPs found. You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/330
-- Commit Summary --
* Fix crash on applications memory reservation
-- File Changes --
M modules/cdp/diameter_avp.c (7) M modules/cdp/peerstatemachine.c (42)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/330.patch https://github.com/kamailio/kamailio/pull/330.diff
--- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/330
Hi,
can you change the commit message according to http://www.kamailio.org/wiki/devel/git-commit-guidelines#commit_message
Thanks, Carstne
--- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/330#issuecomment-141406316
Done,
Thanks to you,
Federico Favaro
--- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/330#issuecomment-141408173
Merged #330.
--- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/330#event-413087334