DTMF begin frames should be ignored so that when an agent acks a call with the '...

he doesn't cause a queue's announce file to be interrupted. Also went ahead and did the
same for the '*' key and for ending a call.

(closes issue #10528, reported by deskhack, patched by me)

git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.4@81120 65c4cc65-6c06-0410-ace0-fbb531ad65f3

diff --git a/channels/chan_agent.c b/channels/chan_agent.c
index 7f1d1cb7c66f50172365f668a6040008ea2dfe67..79c3d230b703699fc7c02df471409e14ff29f8cd 100644 (file)
--- a/channels/chan_agent.c
+++ b/channels/chan_agent.c
@@ -494,6 +494,12 @@ static struct ast_frame *agent_read(struct ast_channel *ast)
                        }
                        break;
                case AST_FRAME_DTMF_BEGIN:
+                       /*ignore DTMF begin's as it can cause issues with queue announce files*/
+                       if((!p->acknowledged && f->subclass == '#') || (f->subclass == '*' && endcall)){
+                               ast_frfree(f);
+                               f = &ast_null_frame;
+                       }
+                       break;
                case AST_FRAME_DTMF_END:
                        if (!p->acknowledged && (f->subclass == '#')) {
                                if (option_verbose > 2)