[asterisk-dev] [Code Review] UNINIT audit and fixes

jrose reviewboard at asterisk.org
Wed May 9 12:51:15 CDT 2012

This is an automatically generated e-mail. To reply, visit:

Review request for Asterisk Developers, Mark Michelson, rmudgett, and Matt Jordan.


This patch addresses most of the UNINIT flagged Coverity issues reported in ASTERISK-19652. The ast_clear_flag macro was a major culprit for a number of them due to the use of bitwise and. There are remarks on each of the reports in checklist_UNINIT in the attached Asterisk issue.

Note:  Coverity specific comments are staying for the time being. This is by the consensus of the dev team since it will make finding these things much easier in the future to properly annotate stuff for Coverity use if that becomes a priority.

This addresses bug ASTERISK-19652.


  /branches/1.8/apps/app_chanspy.c 365630 
  /branches/1.8/apps/app_disa.c 365630 
  /branches/1.8/apps/app_page.c 365630 
  /branches/1.8/funcs/func_cdr.c 365630 
  /branches/1.8/main/features.c 365630 

Diff: https://reviewboard.asterisk.org/r/1909/diff


Tested a little bit for possible garbage values where relevant.



-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20120509/bdea7d02/attachment.htm>

More information about the asterisk-dev mailing list