Send DTMF frame before playing back audio.
authorTilghman Lesher <tilghman@meg.abyt.es>
Thu, 7 May 2009 17:13:36 +0000 (17:13 +0000)
committerTilghman Lesher <tilghman@meg.abyt.es>
Thu, 7 May 2009 17:13:36 +0000 (17:13 +0000)
(closes issue #14858)
 Reported by: barryf
 Patches:
       20090507__bug14858.diff.txt uploaded by tilghman (license 14)

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

channels/chan_unistim.c

index 4706daa..95a8ce8 100644 (file)
@@ -2469,14 +2469,17 @@ static void HandleCallIncoming(struct unistimsession *s)
 
 static int unistim_do_senddigit(struct unistimsession *pte, char digit)
 {
-
-       struct ast_frame f = { 0, };
+       struct ast_frame f = { .frametype = AST_FRAME_DTMF, .subclass = digit, .src = "unistim" };
        struct unistim_subchannel *sub;
        sub = pte->device->lines->subs[SUB_REAL];
        if (!sub->owner || sub->alreadygone) {
                ast_log(LOG_WARNING, "Unable to find subchannel in dtmf senddigit\n");
                return -1;
        }
+
+       /* Send DTMF indication _before_ playing sounds */
+       ast_queue_frame(sub->owner, &f);
+
        if (unistimdebug)
                ast_verb(0, "Send Digit %c\n", digit);
        switch (digit) {
@@ -2533,10 +2536,6 @@ static int unistim_do_senddigit(struct unistimsession *pte, char digit)
        }
        usleep(150000);                  /* XXX Less than perfect, blocking an important thread is not a good idea */
        send_tone(pte, 0, 0);
-       f.frametype = AST_FRAME_DTMF;
-       f.subclass = digit;
-       f.src = "unistim";
-       ast_queue_frame(sub->owner, &f);
        return 0;
 }