[asterisk-dev] [Code Review] Replace nested functions with file scope functions
Sean Bright
sean.bright at gmail.com
Wed Nov 5 19:53:14 CST 2008
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.digium.com/r/33/
-----------------------------------------------------------
Review request for Asterisk Developers.
Summary
-------
This is a patch to promote the current nested functions in asterisk trunk to file level functions based on security concerns. For more details, see:
http://gcc.gnu.org/ml/gcc-help/2005-12/msg00174.html
Also, I added a note to one of the calls in main/features.c which I marked with an XXX comment.
Diffs
-----
/trunk/apps/app_dial.c 154920
/trunk/apps/app_directed_pickup.c 154926
/trunk/apps/app_followme.c 154920
/trunk/apps/app_queue.c 154920
/trunk/channels/chan_sip.c 154920
/trunk/include/asterisk/channel.h 154920
/trunk/main/channel.c 154920
/trunk/main/features.c 154923
Diff: http://reviewboard.digium.com/r/33/diff
Testing
-------
This passes the 'make' test with dev-mode enabled, but I have not functionally tested extensively.
Thanks,
Sean
More information about the asterisk-dev
mailing list