In article <[EMAIL PROTECTED]>, Eric Wieling <[EMAIL PROTECTED]> wrote:
> This is a bounty for a patch to app_hangup.c to generate an error when
> Hangup is called from exten => h.
> 
> You should not call Hangup from exten => h.
> 
> The bounty is US$10 and will be paid via Paypal.  The patch must be
> accepted into CVS-HEAD before the bounty will be paid.

There is no app_hangup.c. The Hangup command is implemented in pbx.c by
the function pbx_builtin_hangup(), which does nothing except return
non-zero. The PBX core then initiates a hangup because the app returned
a non-zero status, which it would for ANY app that did so.

In my opinion the correct fix would be for Asterisk, when an app returns
non-zero, to check if it has already called the h extension for the
channel, and if so, NOT to go to h,1 again. This should be done using
a flag rather than checking the extension, so that it still works if
the dialplan does a Goto out of the h extension to somewhere else.

Cheers
Tony
-- 
Tony Mountifield
Work: [EMAIL PROTECTED] - http://www.softins.co.uk
Play: [EMAIL PROTECTED] - http://tony.mountifield.org
_______________________________________________
Asterisk-Dev mailing list
[email protected]
http://lists.digium.com/mailman/listinfo/asterisk-dev
To UNSUBSCRIBE or update options visit:
   http://lists.digium.com/mailman/listinfo/asterisk-dev

Reply via email to