]> 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:15:09 +0000 (21:15 +0000)
committerMark Michelson <mmichelson@digium.com>
Mon, 15 Oct 2012 21:15:09 +0000 (21:15 +0000)
commit94c0fa90980f4223babc41a9f734cf25ad620375
tree37bc31f367b835f21e896ea84b300a7df997c5d1
parent5b1a89e1b16657e577e05e8b8232efef7d86bae7
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
........

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

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