[OpenSIPS-Users] [OpenSIPS Security Alerts] [FIX] [Severity High] ACC module

[Vlad Paiu]

This message was generated by the Security Alerts service ( Free Trial 14th of August - 14th of September )
http://www.opensips.org/Resources/AlertsMain
*
SVN commit*:
http://opensips.svn.sourceforge.net/viewvc/opensips?view=revision&revision=9195

*Severity*: High

*Version*  : 1.8, trunk

*Affected modules*  : ACC

*Effect*  : Missing CDRs in the back-end

*Affected scenarios*: The issue appeared when the ACC module was set to generate proper
CDRs with the duration field also populated ( cdr_flag ). When also setting the
db_extra_bye parameter in order to have some extra accounting for variable evaluated at
BYE time, internally ended dialogs ( expired dialogs or forcefully terminated on the
OpenSIPS side via an MI command ) would not generate accounting records.

*Description:*  The bug was introduced along with the db_extra_bye feature, due to an
incorrectly treated error case.
In order to evaluate the pseudo-variables set in the db_extra_bye parameter, OpenSIPS
requires the BYE SIP message pointer  Because in the case of internally terminated dialogs
there is no actual SIP message received from the network level, OpenSIPS would fail to
evaluate the pseudo-variables, and then due to improper error handling, the accounting
process was stopped, and no CDR was generated in the back-end.

*Risks*  : When doing the new type of accounting with CDRs, there was the risk of having
some missing CDRs, and thus if using CDR type accounting, upgrading is critical.

*Update*  :
- if you have an SVN checkout, 1.8 and trunk were fixed; so
update to a revision later than 9194 (trunk) or 9195 (1.8 branch)
- if you have OpenSIPS from sources, download and apply the patch from
http://opensips.svn.sourceforge.net/viewvc/opensips?view=revision&revision=9195
or see the attached patch;
- if using tarballs, they were already regenerated (and include the fix)
- If using the official Debian package (apt.opensips.org), they are also
re-generated including the fix.

-- 
Vlad Paiu
OpenSIPS Developer
http://www.opensips-solutions.com

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.opensips.org/pipermail/users/attachments/20120827/92ce3e4b/attachment.htm>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: acc_cdrs-9195.patch
Type: text/x-diff
Size: 859 bytes
Desc: not available
URL: <http://lists.opensips.org/pipermail/users/attachments/20120827/92ce3e4b/attachment.patch>
-------------- next part --------------
_______________________________________________
Alerts mailing list
Alerts at lists.opensips.org
http://lists.opensips.org/cgi-bin/mailman/listinfo/alerts