diff --git a/codecs/codec_a_mu.c b/codecs/codec_a_mu.c index 51e38e9cf67e9691fd44289ff29a8b9722366ed4..6ab79be61418b1ad0ce146d6e297addba2d48340 100644 --- a/codecs/codec_a_mu.c +++ b/codecs/codec_a_mu.c @@ -55,28 +55,30 @@ static unsigned char a2mu[256]; /*! \brief convert frame data and store into the buffer */ static int alawtoulaw_framein(struct ast_trans_pvt *pvt, struct ast_frame *f) { - int x; + int i; + int in_samples = f->samples; unsigned char *src = f->data; unsigned char *dst = (unsigned char *)pvt->outbuf + pvt->samples; - for ( x = 0 ; x < f->samples; x++) - dst[x] = a2mu[src[x]]; - pvt->samples += f->samples; - pvt->datalen += f->datalen; + for (i = 0; i < in_samples; i++) + *dst++ = a2mu[*src++]; + pvt->samples += in_samples; + pvt->datalen += in_samples; return 0; } /*! \brief convert frame data and store into the buffer */ static int ulawtoalaw_framein(struct ast_trans_pvt *pvt, struct ast_frame *f) { - int x; + int i; + int in_samples = f->samples; unsigned char *src = f->data; unsigned char *dst = (unsigned char *)pvt->outbuf + pvt->samples; - for ( x = 0 ; x < f->samples; x++) - dst[x] = mu2a[src[x]]; - pvt->samples += f->samples; - pvt->datalen += f->datalen; + for (i = 0; i < in_samples; i++) + *dst++ = mu2a[*src++]; + pvt->samples += in_samples; + pvt->datalen += in_samples; return 0; } diff --git a/codecs/codec_alaw.c b/codecs/codec_alaw.c index d654e21e4838cd78b95fe836d68c486c3723ab96..8ca185f750db2737f1b9f87af3c5801a7aab862d 100644 --- a/codecs/codec_alaw.c +++ b/codecs/codec_alaw.c @@ -57,25 +57,29 @@ static int alawtolin_framein(struct ast_trans_pvt *pvt, struct ast_frame *f) int i; unsigned char *src = f->data; int16_t *dst = (int16_t *)pvt->outbuf; - - for ( i = 0; i < f->samples; i++) - dst[pvt->samples + i] = AST_ALAW(src[i]); - pvt->samples += f->samples; - pvt->datalen += 2*f->samples; /* 2 bytes/sample */ + int in_samples = f->samples; + int out_samples = pvt->samples; + + for (i = 0; i < in_samples; i++) + dst[out_samples++] = AST_ALAW(src[i]); + + pvt->samples = out_samples; + pvt->datalen += in_samples * 2; /* 2 bytes/sample */ return 0; } /*! \brief convert and store input samples in output buffer */ static int lintoalaw_framein(struct ast_trans_pvt *pvt, struct ast_frame *f) { - int x; + int i; char *dst = pvt->outbuf + pvt->samples; int16_t *src = f->data; + int in_samples = f->samples; - for ( x = 0; x < f->samples; x++) - *dst++ = AST_LIN2A(src[x]); - pvt->samples += f->samples; - pvt->datalen += f->samples; /* 1 byte/sample */ + for (i = 0; i < in_samples; i++) + *dst++ = AST_LIN2A(src[i]); + pvt->samples += in_samples; + pvt->datalen += in_samples; /* 1 byte/sample */ return 0; } diff --git a/codecs/codec_ulaw.c b/codecs/codec_ulaw.c index 0db01de7ca0774d33495ee975aaa3dcf237c8e59..41eb5b30858dc66c5191921b1608cbe875d7960b 100644 --- a/codecs/codec_ulaw.c +++ b/codecs/codec_ulaw.c @@ -57,13 +57,15 @@ static int ulawtolin_framein(struct ast_trans_pvt *pvt, struct ast_frame *f) int i; unsigned char *src = f->data; int16_t *dst = (int16_t *)pvt->outbuf; + int in_samples = f->samples; + int out_samples = pvt->samples; /* convert and copy in outbuf */ - for (i=0; i<f->samples; i++) - dst[pvt->samples + i] = AST_MULAW(src[i]); + for (i = 0; i < in_samples; i++) + dst[out_samples++] = AST_MULAW(src[i]); - pvt->samples += f->samples; - pvt->datalen += 2 * f->samples; + pvt->samples = out_samples; + pvt->datalen += in_samples * 2; /* 2 bytes/sample */ return 0; } @@ -71,12 +73,14 @@ static int ulawtolin_framein(struct ast_trans_pvt *pvt, struct ast_frame *f) static int lintoulaw_framein(struct ast_trans_pvt *pvt, struct ast_frame *f) { int i; + char *dst = pvt->outbuf + pvt->samples; int16_t *src = f->data; + int in_samples = f->samples; - for (i=0 ; i < f->samples; i++) - pvt->outbuf[pvt->samples + i] = AST_LIN2MU(src[i]); - pvt->samples += f->samples; - pvt->datalen += f->samples; /* 1 byte/sample */ + for (i = 0; i < in_samples; i++) + *dst++ = AST_LIN2MU(src[i]); + pvt->samples += in_samples; + pvt->datalen += in_samples; /* 1 byte/sample */ return 0; } @@ -147,7 +151,7 @@ static void parse_config(void) { struct ast_variable *var; struct ast_config *cfg = ast_config_load("codecs.conf"); - if (cfg == NULL) + if (!cfg) return; for (var = ast_variable_browse(cfg, "plc"); var; var = var->next) { if (!strcasecmp(var->name, "genericplc")) {