]> git.ipfire.org Git - thirdparty/asterisk.git/commit
Make use of va_args more appropriate to form in various res_config modules plus utils.
authorJonathan Rose <jrose@digium.com>
Tue, 17 Apr 2012 20:43:22 +0000 (20:43 +0000)
committerJonathan Rose <jrose@digium.com>
Tue, 17 Apr 2012 20:43:22 +0000 (20:43 +0000)
commitbcd63be3cdc21f6733411863685c08c53ca9bf0c
tree3c858fce7eb9623bb5630b4b493d834fd31a3444
parent8cbe9f9fa764b79d146acfe80c7243d24541762f
Make use of va_args more appropriate to form in various res_config modules plus utils.

A number of va_copy operations weren't matched with a corresponding va_end in res_config_odbc. Also, there was a potential for va_end to be invoked twice on the same va_arg in utils, which would mean invoking va_end on an undefined variable... which is bad.
va_end is removed from various functions in config_pgsql and config_curl since they aren't making their own copy.  The invokers of those functions are responsible for calling va_end on them.

(issue ASTERISK-19451)
Reported by: Walter Doekes
Review: https://reviewboard.asterisk.org/r/1848/

git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.8@362354 65c4cc65-6c06-0410-ace0-fbb531ad65f3
main/utils.c
res/res_config_curl.c
res/res_config_odbc.c
res/res_config_pgsql.c