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 21:10:50 +0000 (21:10 +0000)
committerJonathan Rose <jrose@digium.com>
Tue, 17 Apr 2012 21:10:50 +0000 (21:10 +0000)
commitf88a632d96bb3ed76976b2f32f5d7b7e8591afa9
treeaf95e26373900d175d853edbc817658fa205acbe
parent3934b0478d4962a81fcc5d20b75dbfc3aceb398e
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/
........

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

Merged revisions 362357 from http://svn.asterisk.org/svn/asterisk/branches/10

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@362363 65c4cc65-6c06-0410-ace0-fbb531ad65f3
res/res_config_curl.c
res/res_config_odbc.c
res/res_config_pgsql.c