Change SHARED function to use a safe traversal when modifying a variable
authorMatthew Jordan <mjordan@digium.com>
Mon, 9 Apr 2012 19:44:35 +0000 (19:44 +0000)
committerMatthew Jordan <mjordan@digium.com>
Mon, 9 Apr 2012 19:44:35 +0000 (19:44 +0000)
commitf4fd1b2fb0267991ecc1e2183d5a5833d0a6c4a3
tree2c7df830257a978a7dd362690554c04c3bf167c7
parent90226b6fd7e21eb149c8def79596247ed2366b6e
Change SHARED function to use a safe traversal when modifying a variable

When the SHARED function modifies a variable, it removes it from its list of
variables and reinserts the new value at the head of the list of variables.
Doing this inside a standard list traversal can be dangerous, as the
standard list traversal does not account for the list being changed.  While
the code in question should not cause a use after free violation due to its
breaking out of the loop after freeing the variable, it could lead to a
maintenance issue if the loop was modified.  This also fixes a violation
reported by a static analysis tool, which also makes this code easier to
maintain in the future.
........

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

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

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@361659 65c4cc65-6c06-0410-ace0-fbb531ad65f3
funcs/func_global.c