[Asterisk-code-review] core: Add AO2_ALLOC_OPT_NO_REF_DEBUG option. (...asterisk[master])

Corey Farrell asteriskteam at digium.com
Mon Sep 23 12:33:46 CDT 2019


Hello Richard Mudgett, Friendly Automation, 

I'd like you to reexamine a change. Please visit

    https://gerrit.asterisk.org/c/asterisk/+/12912

to look at the new patch set (#3).

Change subject: core: Add AO2_ALLOC_OPT_NO_REF_DEBUG option.
......................................................................

core: Add AO2_ALLOC_OPT_NO_REF_DEBUG option.

Previous to this patch passing a NULL tag to ao2_alloc or ao2_ref based
functions would result in the reference not being logged under
REF_DEBUG.  This could sometimes cause inaccurate logging if NULL was
accidentally passed to a reference action.  Now reference logging is
only disabled by option passed to the allocation method.

Change-Id: I3c17d867d901d53f9fcd512bef4d52e342637b54
---
M include/asterisk/astobj2.h
M main/astobj2.c
M main/astobj2_container.c
M main/astobj2_hash.c
M main/astobj2_rbtree.c
5 files changed, 71 insertions(+), 70 deletions(-)


  git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/12/12912/3
-- 
To view, visit https://gerrit.asterisk.org/c/asterisk/+/12912
To unsubscribe, or for help writing mail filters, visit https://gerrit.asterisk.org/settings

Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-Change-Id: I3c17d867d901d53f9fcd512bef4d52e342637b54
Gerrit-Change-Number: 12912
Gerrit-PatchSet: 3
Gerrit-Owner: Corey Farrell <git at cfware.com>
Gerrit-Reviewer: Corey Farrell <git at cfware.com>
Gerrit-Reviewer: Friendly Automation
Gerrit-Reviewer: Richard Mudgett <rmudgett at digium.com>
Gerrit-MessageType: newpatchset
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20190923/873eb649/attachment-0001.html>


More information about the asterisk-code-review mailing list