|Summary:||ASTERISK-06653: [patch] minor change and code clean in app_stack.c|
|Reporter:||Peng Yong (ppyy)||Labels:|
|Date Opened:||2006-03-29 08:19:49.000-0600||Date Closed:||2006-03-29 13:34:37.000-0600|
|Environment:||Attachments:||( 0) patch.app_stack|
( 1) patch.app_stack.2
|Description:||1. remove LOCAL_USER_ADD in gosub_exec, it will called in the gosub_exec which is called by gosubif_exec|
2. code clean
|Comments:||By: Tilghman Lesher (tilghman) 2006-03-29 08:30:33.000-0600|
You obviously did not test this patch, because it does not work.
By: Peng Yong (ppyy) 2006-03-29 08:38:28.000-0600
sorry, upload again
By: Tilghman Lesher (tilghman) 2006-03-29 13:34:11.000-0600
The point of LOCAL_USER_ADD is that we don't attempt to unload while executing. This is why calling it twice is not harmful, but leaving it wide open could be. I have to disagree with your patch for this reason.