diff options
author | Kinsey Moore <kmoore@digium.com> | 2013-01-21 18:47:47 +0000 |
---|---|---|
committer | Kinsey Moore <kmoore@digium.com> | 2013-01-21 18:47:47 +0000 |
commit | 8e3815a8ee9db8d7750223626c393f1a96e4ff3a (patch) | |
tree | 88b174558f7a1bcfd45a9fafa2e1929064a7defb /codecs | |
parent | b5962bd5f671053beb5af99e3dfaf72e9071b21c (diff) |
Prevent segfault for interpolated iLBC frames
When iLBC is being used with a jitter buffer and the jb has to
interpolate frames, it generates frames with a null pointer and a
non-zero datalen. This is now handled properly.
(closes issue ASTERISK-20914)
Reported By: John McEleney
Patches:
ASTERISK-20914-1.8.diff uploaded by Matt Jordan (license 6283)
........
Merged revisions 379718 from http://svn.asterisk.org/svn/asterisk/branches/1.8
........
Merged revisions 379719 from http://svn.asterisk.org/svn/asterisk/branches/11
git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@379721 65c4cc65-6c06-0410-ace0-fbb531ad65f3
Diffstat (limited to 'codecs')
-rw-r--r-- | codecs/codec_ilbc.c | 16 |
1 files changed, 9 insertions, 7 deletions
diff --git a/codecs/codec_ilbc.c b/codecs/codec_ilbc.c index dacdd1eff..632169589 100644 --- a/codecs/codec_ilbc.c +++ b/codecs/codec_ilbc.c @@ -96,27 +96,29 @@ static int ilbctolin_framein(struct ast_trans_pvt *pvt, struct ast_frame *f) /* Assuming there's space left, decode into the current buffer at the tail location. Read in as many frames as there are */ int x,i; + int datalen = f->datalen; int16_t *dst = pvt->outbuf.i16; ilbc_block tmpf[ILBC_SAMPLES]; - if (!f->data.ptr && f->datalen) { - ast_debug(1, "issue 16070, ILIB ERROR. data = NULL datalen = %d src = %s\n", f->datalen, f->src ? f->src : "no src set"); + if (!f->data.ptr && datalen) { + ast_debug(1, "issue 16070, ILIB ERROR. data = NULL datalen = %d src = %s\n", datalen, f->src ? f->src : "no src set"); f->datalen = 0; + datalen = 0; } - if (f->datalen == 0) { /* native PLC, set fake f->datalen and clear plc_mode */ - f->datalen = ILBC_FRAME_LEN; + if (datalen == 0) { /* native PLC, set fake datalen and clear plc_mode */ + datalen = ILBC_FRAME_LEN; f->samples = ILBC_SAMPLES; plc_mode = 0; /* do native plc */ pvt->samples += ILBC_SAMPLES; } - if (f->datalen % ILBC_FRAME_LEN) { - ast_log(LOG_WARNING, "Huh? An ilbc frame that isn't a multiple of 50 bytes long from %s (%d)?\n", f->src, f->datalen); + if (datalen % ILBC_FRAME_LEN) { + ast_log(LOG_WARNING, "Huh? An ilbc frame that isn't a multiple of 50 bytes long from %s (%d)?\n", f->src, datalen); return -1; } - for (x=0; x < f->datalen ; x += ILBC_FRAME_LEN) { + for (x=0; x < datalen ; x += ILBC_FRAME_LEN) { if (pvt->samples + ILBC_SAMPLES > BUFFER_SAMPLES) { ast_log(LOG_WARNING, "Out of buffer space\n"); return -1; |