Merged revisions 313780 via svnmerge from
authorRichard Mudgett <rmudgett@digium.com>
Thu, 14 Apr 2011 21:02:38 +0000 (21:02 +0000)
committerRichard Mudgett <rmudgett@digium.com>
Thu, 14 Apr 2011 21:02:38 +0000 (21:02 +0000)
https://origsvn.digium.com/svn/asterisk/branches/1.8

........
  r313780 | rmudgett | 2011-04-14 15:59:56 -0500 (Thu, 14 Apr 2011) | 20 lines

  Leftover debug messages unconditionally sent to the console.

  Executing Dial(DAHDI/1/18475551212,300,) with the echotraining config
  option enabled outputs the following debug messages unconditionally:

  Dialing T1847555121 on 1
  Dialing www2w on 1

  * Made debug messages in my_dial_digits() normal debug messages that do
  not get output unless enabled.

  * Reworded some debug messages in my_dial_digits() to be clearer.

  * Replace strncpy() with ast_copy_string() in my_dial_digits() which does
  the same job better.

  (closes issue #18847)
  Reported by: vmikhelson
  Tested by: rmudgett
........

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

channels/chan_dahdi.c

index bbb5c1a..b11d114 100644 (file)
@@ -2943,18 +2943,21 @@ static int my_dial_digits(void *pvt, enum analog_sub sub, struct analog_dialoper
                return -1;
        }
 
-       if (sub != ANALOG_SUB_REAL)
-               printf("Trying to dial digits on sub %d\n", sub);
+       if (sub != ANALOG_SUB_REAL) {
+               ast_log(LOG_ERROR, "Trying to dial_digits '%s' on channel %d subchannel %d\n",
+                       dop->dialstr, p->channel, sub);
+               return -1;
+       }
 
        ddop.op = DAHDI_DIAL_OP_REPLACE;
-       strncpy(ddop.dialstr, dop->dialstr, sizeof(ddop.dialstr));
+       ast_copy_string(ddop.dialstr, dop->dialstr, sizeof(ddop.dialstr));
 
-       printf("Dialing %s on %d\n", ddop.dialstr, p->channel);
+       ast_debug(1, "Channel %d: Sending '%s' to DAHDI_DIAL.\n", p->channel, ddop.dialstr);
 
        res = ioctl(p->subs[index].dfd, DAHDI_DIAL, &ddop);
-
-       if (res == -1)
+       if (res == -1) {
                ast_debug(1, "DAHDI_DIAL ioctl failed on %s: %s\n", p->owner->name, strerror(errno));
+       }
 
        return res;
 }