]> git.ipfire.org Git - thirdparty/asterisk.git/commit
Fix some potential misuses of ast_str in the code.
authorMark Michelson <mmichelson@digium.com>
Mon, 15 Oct 2012 21:06:42 +0000 (21:06 +0000)
committerMark Michelson <mmichelson@digium.com>
Mon, 15 Oct 2012 21:06:42 +0000 (21:06 +0000)
commit3e2a2ac24fea52b2bb6f80e906d45e6ae6f645ff
tree3fcae8d057f204e1e94f151fd1baa731581236e0
parent46862eee6601f5f071a2c2486181fab0ba0920da
Fix some potential misuses of ast_str in the code.

Passing an ast_str pointer by value that then calls
ast_str_set(), ast_str_set_va(), ast_str_append(), or
ast_str_append_va() can result in the pointer originally
passed by value being invalidated if the ast_str had
to be reallocated.

This fixes places in the code that do this. Only the
example in ccss.c could result in pointer invalidation
though since the other cases use a stack-allocated ast_str
and cannot be reallocated.

I've also updated the doxygen in strings.h to include
notes about potential misuse of the functions mentioned
previously.

Review: https://reviewboard.asterisk.org/r/2161
........

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

git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/10@375026 65c4cc65-6c06-0410-ace0-fbb531ad65f3
apps/app_dial.c
channels/chan_iax2.c
include/asterisk/strings.h
main/ccss.c