dect
/
asterisk
Archived
13
0
Fork 0
Code Releases Activity

codecs/codec_dahdi: Prevent "choppy" audio when receiving unexpected frame sizes. 

dahdi-linux 2.4.0 (specifically commit 9034) added the capability for
the wctc4xxp to return more than a single packet of data in response to
a read.  However, when decoding packets, codec_dahdi was still assuming
that the default number of samples was in each read.

In other words, each packet your provider sent you, regardless of size,
would result in 20 ms of decoded data (30 ms if decoding G723). If your
provider was sending 60 ms packets then codec_dahdi would end up
stripping 40 ms of data from each transcoded frame resulting in "choppy"
audio.

This would only affect systems where G729 packets are arriving in sizes
greater than 20ms or G723 packets arriving in sizes greater than 30ms.

DAHDI-744.

git-svn-id: http://svn.digium.com/svn/asterisk/branches/1.4@293968 f38db490-d61c-443f-a65b-d21fe96a405b
This commit is contained in:
sruffell 2010-11-05 00:02:53 +00:00
parent 44d53b8aa7
commit 859dac960e
1 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
codecs/codec_dahdi.c
View File

@ -120,10 +120,10 @@ struct codec_dahdi_pvt {
};
/* Only used by a decoder */
static int ulawtolin(struct ast_trans_pvt *pvt)
static int ulawtolin(struct ast_trans_pvt *pvt, int samples)
{
struct codec_dahdi_pvt *ztp = pvt->pvt;
int i = ztp->required_samples;
int i = samples;
uint8_t *src = &ztp->ulaw_buffer[0];
int16_t *dst = (int16_t *)pvt->outbuf + pvt->datalen;
@ -345,7 +345,7 @@ static struct ast_frame *dahdi_decoder_frameout(struct ast_trans_pvt *pvt)
}
} else {
if (ztp->softslin) {
ulawtolin(pvt);
ulawtolin(pvt, res);
pvt->f.datalen = res * 2;
} else {
pvt->f.datalen = res;
@ -357,7 +357,7 @@ static struct ast_frame *dahdi_decoder_frameout(struct ast_trans_pvt *pvt)
pvt->f.offset = AST_FRIENDLY_OFFSET;
pvt->f.src = pvt->t->name;
pvt->f.data = pvt->outbuf;
pvt->f.samples = ztp->required_samples;
pvt->f.samples = res;
pvt->samples = 0;
return ast_frisolate(&pvt->f);