[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