Hello, On 11/6/06, Andrzej Ciarkowski <[EMAIL PROTECTED]> wrote: > I was expecting the same problems - it happened when I received some > none-standard length G711 frames from one of the operators around > (AFAIR this was callcentric.com). The problem is that jitter buffer > queue size is not an integer multiply of received frame length and > RTP data overwrites the buffer. The attached patches solve this > problem allowing the frame lenghts of arbitrary size and preventing > buffer boundary overwrites. Thanks for your patches, it's always helpful to see fix for reported bug. :) But I think solution should be slightly other way. One possible way is to set JitterBuffer size dynamically on creation. The other way is to push packet into decoder and then get decoded data in several chunks.
> I don't know how to deal with buffer underruns, though, so I left the current > solution - contents of output buffer is untouched. We should implement Packet Loss Concealment here. Some codecs have their own PLCs, but other have not. So we should implement generic one and leave possibility for codec-specific PLC. Jitter Buffer and PLC are the one of most complicated parts of VoIP and at the same time very important for good sound quality. -- Regards, Alexander Chemeris. _______________________________________________ sipxtapi-dev mailing list [email protected] List Archive: http://list.sipfoundry.org/archive/sipxtapi-dev/
