[asterisk-commits] russell: branch 10 r359454 - in /branches/10: ./ include/asterisk/app.h

SVN commits to the Asterisk project asterisk-commits at lists.digium.com
Wed Mar 14 17:37:05 CDT 2012


Author: russell
Date: Wed Mar 14 17:37:01 2012
New Revision: 359454

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=359454
Log:
app.h: Always initialize AST_DECLARE_APP_ARGS().

This patch ensures that the struct defined by AST_DECLARE_APP_ARGS() is always
fully initialized.  I'm not sure if this fixes any real bugs, but it silences
a bunch of warnings from coverity, and is generally a good thing to do anyway.
........

Merged revisions 359452 from http://svn.asterisk.org/svn/asterisk/branches/1.8

Modified:
    branches/10/   (props changed)
    branches/10/include/asterisk/app.h

Propchange: branches/10/
------------------------------------------------------------------------------
Binary property 'branch-1.8-merged' - no diff available.

Modified: branches/10/include/asterisk/app.h
URL: http://svnview.digium.com/svn/asterisk/branches/10/include/asterisk/app.h?view=diff&rev=359454&r1=359453&r2=359454
==============================================================================
--- branches/10/include/asterisk/app.h (original)
+++ branches/10/include/asterisk/app.h Wed Mar 14 17:37:01 2012
@@ -432,7 +432,7 @@
   ast_app_separate_args() will perform that function before parsing
   the arguments.
  */
-#define AST_DECLARE_APP_ARGS(name, arglist) AST_DEFINE_APP_ARGS_TYPE(, arglist) name
+#define AST_DECLARE_APP_ARGS(name, arglist) AST_DEFINE_APP_ARGS_TYPE(, arglist) name = { 0, }
 
 /*!
   \brief Define a structure type to hold an application's arguments.




More information about the asterisk-commits mailing list