Merged revisions 229281 via svnmerge from
authorJoshua Colp <jcolp@digium.com>
Tue, 10 Nov 2009 20:06:13 +0000 (20:06 +0000)
committerJoshua Colp <jcolp@digium.com>
Tue, 10 Nov 2009 20:06:13 +0000 (20:06 +0000)
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
  r229281 | file | 2009-11-10 16:03:14 -0400 (Tue, 10 Nov 2009) | 8 lines

  Remove broken support for direct transcoding between G.726 RFC3551 and G.726 AAL2.

  On some systems the translation core would actually consider g726aal2 -> g726 -> signed linear
  to be a quicker path then g726aal2 -> signed linear which exposed this problem.

  (closes issue #15504)
  Reported by: globalnetinc
........

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

codecs/codec_g726.c

index cce448f..b02bae0 100644 (file)
@@ -769,22 +769,6 @@ static int lintog726_framein(struct ast_trans_pvt *pvt, struct ast_frame *f)
        return 0;
 }
 
-/*! \brief convert G726-32 RFC3551 packed data into AAL2 packed data (or vice-versa) */
-static int g726tog726aal2_framein(struct ast_trans_pvt *pvt, struct ast_frame *f)
-{
-       unsigned char *src = f->data.ptr;
-       unsigned char *dst = pvt->outbuf.uc + pvt->samples;
-       unsigned int i;
-
-       for (i = 0; i < f->datalen; i++)
-               *dst++ = ((src[i] & 0xf) << 4) | (src[i] >> 4);
-
-       pvt->samples += f->samples;
-       pvt->datalen += f->samples; /* 1 byte/sample */
-
-       return 0;
-}
-
 static struct ast_translator g726tolin = {
        .name = "g726tolin",
        .srcfmt = AST_FORMAT_G726,
@@ -835,26 +819,6 @@ static struct ast_translator lintog726aal2 = {
        .buf_size = BUFFER_SAMPLES / 2,
 };
 
-static struct ast_translator g726tog726aal2 = {
-       .name = "g726tog726aal2",
-       .srcfmt = AST_FORMAT_G726,
-       .dstfmt = AST_FORMAT_G726_AAL2,
-       .framein = g726tog726aal2_framein,      /* same for both directions */
-       .sample = g726_sample,
-       .buffer_samples = BUFFER_SAMPLES,
-       .buf_size = BUFFER_SAMPLES,
-};
-
-static struct ast_translator g726aal2tog726 = {
-       .name = "g726aal2tog726",
-       .srcfmt = AST_FORMAT_G726_AAL2,
-       .dstfmt = AST_FORMAT_G726,
-       .framein = g726tog726aal2_framein,      /* same for both directions */
-       .sample = g726_sample,
-       .buffer_samples = BUFFER_SAMPLES,
-       .buf_size = BUFFER_SAMPLES,
-};
-
 static int parse_config(int reload)
 {
        struct ast_variable *var;
@@ -891,9 +855,6 @@ static int unload_module(void)
        res |= ast_unregister_translator(&g726aal2tolin);
        res |= ast_unregister_translator(&lintog726aal2);
 
-       res |= ast_unregister_translator(&g726aal2tog726);
-       res |= ast_unregister_translator(&g726tog726aal2);
-
        return res;
 }
 
@@ -911,9 +872,6 @@ static int load_module(void)
        res |= ast_register_translator(&g726aal2tolin);
        res |= ast_register_translator(&lintog726aal2);
 
-       res |= ast_register_translator(&g726aal2tog726);
-       res |= ast_register_translator(&g726tog726aal2);
-
        if (res) {
                unload_module();
                return AST_MODULE_LOAD_FAILURE;