[Home]

Summary:ASTERISK-26171: REF_DEBUG: support writing storage address to refs log.
Reporter:Corey Farrell (coreyfarrell)Labels:
Date Opened:2016-07-02 14:55:28Date Closed:2017-02-21 14:53:47.000-0600
Priority:MinorRegression?
Status:Closed/CompleteComponents:Core/AstMM
Versions:SVN Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) refs-filter-matched.txt
( 1) refs-unfiltered.txt
( 2) refs-updated.txt
Description:The ability to associate an AO2 reference with a pointer will allow {{refcounter.py}} to greatly reduce the output for leaked objects.  The use of ast_format in frame.c is a perfect example.  The same format is often used thousands of times but frame.c is rarely the cause of a format leaking.  This change causes matching ref/unref from frame.c and a few other places to be filtered out of the results.  This is done by associating references with address of the pointer that will hold it.
Comments:By: Asterisk Team (asteriskteam) 2016-07-02 14:55:29.410-0500

Thanks for creating a report! The issue has entered the triage process. That means the issue will wait in this status until a Bug Marshal has an opportunity to review the issue. Once the issue has been reviewed you will receive comments regarding the next steps towards resolution.

A good first step is for you to review the [Asterisk Issue Guidelines|https://wiki.asterisk.org/wiki/display/AST/Asterisk+Issue+Guidelines] if you haven't already. The guidelines detail what is expected from an Asterisk issue report.

Then, if you are submitting a patch, please review the [Patch Contribution Process|https://wiki.asterisk.org/wiki/display/AST/Patch+Contribution+Process].

By: Corey Farrell (coreyfarrell) 2016-07-02 14:58:55.866-0500

Attached is output of my updated {{refcounter.py}} for an execution of testsuite {{tests/bridge/hold_redirect}}.  One shows all references to leaked objects, the other only shows unmatched references to leaked objects.

By: Corey Farrell (coreyfarrell) 2016-07-08 13:31:01.960-0500

refs-updated.txt contains a more recent run of {{tests/bridge/hold_redirect}} showing detection of indirect leaks.