Hello *,
We've run into a problem with the gstreamer sbcdec plugin, where a
pipeline would accumulate memory at the rate of data sent into it. This
is caused by invalid data in a packet (incomplete or wrong data in the
stream), causing sbc_decode to return -2 (sbc_unpack_frame can't find
the SBC_SYNCWORD in data[0]).
The sbcdec plugin handles this by putting away the buffer, and trying
again when a new buffer comes in. Since the buffer isn't changed (only
new data tacked on), sbc_decode will continue to send the -2 response
from sbc_unpack_frame function, and the sbcdec plugin will continue to
accumulate data into the buffer it is working on.
The fix is to simply drop a buffer when sbc_decode returns -2.
Regards,
Bas Vermeulen
--
novero gmbh | parsevalstr. 7 a | 40468 düsseldorf | germany |
amtsgericht düsseldorf | hrb 58283 | umsatzsteueridentifikationsnummer:
de 814973142 | geschäftsführender gesellschafter: razvan olosu
--- /usr/src/nokia/mms_p2430-mute/mms_p2430/src/plugins/audio/decoder/gstrtpsbc/src/gstsbcdec.c 2009-12-01 11:32:45.000000000 +0100
+++ gstsbcdec.c 2009-12-04 10:06:34.000000000 +0100
@@ -95,7 +95,12 @@
GST_BUFFER_DATA(output), codesize,
NULL);
if (consumed <= 0)
- break;
+ {
+ if (-2 == consumed)
+ goto done;
+ else
+ break;
+ }
/* we will reuse the same caps object */
if (dec->outcaps == NULL) {
