[asterisk-dev] [Code Review] 3971: astobj2.c/refcounter.py: Fix to deal with invalid object refs.

[Corey Farrell]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/3971/#review13307
-----------------------------------------------------------



/branches/1.8/contrib/scripts/refcounter.py
<https://reviewboard.asterisk.org/r/3971/#comment23786>

    This can be optimized if invalid objects are given curcount == -1, that way it wouldn't have to do substring comparison again.


- Corey Farrell


On Sept. 2, 2014, 10:22 p.m., rmudgett wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/3971/
> -----------------------------------------------------------
> 
> (Updated Sept. 2, 2014, 10:22 p.m.)
> 
> 
> Review request for Asterisk Developers.
> 
> 
> Bugs: AFS-155 and ASTERISK-24295
>     https://issues.asterisk.org/jira/browse/AFS-155
>     https://issues.asterisk.org/jira/browse/ASTERISK-24295
> 
> 
> Repository: Asterisk
> 
> 
> Description
> -------
> 
> * Make astob2 REF_DEBUG output an invalid object line when an invalid ao2 object ref/unref is attempted.  This is similar to the constructor/destructor lines.
> 
> * Fixed refcounter.py to handle skewed objects that have constructor/destructor states.
> 
> * Made refcounter.py highlight the invalid ao2 object refs by putting them in their own section of the processed output file.
> 
> 
> Diffs
> -----
> 
>   /branches/1.8/main/astobj2.c 422535 
>   /branches/1.8/contrib/scripts/refcounter.py 422535 
> 
> Diff: https://reviewboard.asterisk.org/r/3971/diff/
> 
> 
> Testing
> -------
> 
> Instigated a REF_DEBUG log that reffed destroyed objects to get an invalid line in the ref debug log file.
> The patched script doesn't have an excpetion on a previous invalid ref debug log that had code unreffing a destroyed object.
> The patched script now highlights the invalid objects in the log.
> 
> 
> Thanks,
> 
> rmudgett
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20140915/b6c5018e/attachment-0001.html>