static FrameHeaderCheckReturn
gst_flac_parse_frame_header_is_valid (GstFlacParse * flacparse,
- const guint8 * data, guint size, gboolean set, guint16 * block_size_ret)
+ const guint8 * data, guint size, gboolean set, guint16 * block_size_ret,
+ gboolean * suspect)
{
GstBitReader reader = GST_BIT_READER_INIT (data, size);
guint8 blocking_strategy;
/* TODO: can we know we're on the last frame, to avoid warning ? */
GST_WARNING_OBJECT (flacparse, "Block size is not constant");
block_size = flacparse->block_size;
+ if (suspect)
+ *suspect = TRUE;
}
}
}
guint i, search_start, search_end;
FrameHeaderCheckReturn header_ret;
guint16 block_size;
+ gboolean suspect_start = FALSE, suspect_end = FALSE;
buffer = frame->buffer;
data = GST_BUFFER_DATA (buffer);
header_ret =
gst_flac_parse_frame_header_is_valid (flacparse, data, size, TRUE,
- &block_size);
+ &block_size, &suspect_start);
if (header_ret == FRAME_HEADER_INVALID) {
*ret = 0;
return FALSE;
for (i = search_start; i < search_end; i++, remaining--) {
if ((GST_READ_UINT16_BE (data + i) & 0xfffe) == 0xfff8) {
+ GST_LOG_OBJECT (flacparse, "possible frame end at offset %d", i);
+ suspect_end = FALSE;
header_ret =
gst_flac_parse_frame_header_is_valid (flacparse, data + i, remaining,
- FALSE, NULL);
+ FALSE, NULL, &suspect_end);
if (header_ret == FRAME_HEADER_VALID) {
- if (flacparse->check_frame_checksums) {
+ if (flacparse->check_frame_checksums || suspect_start || suspect_end) {
guint16 actual_crc = gst_flac_calculate_crc16 (data, i - 2);
guint16 expected_crc = GST_READ_UINT16_BE (data + i - 2);
- if (actual_crc != expected_crc)
+ GST_LOG_OBJECT (flacparse,
+ "checking checksum, frame suspect (%d, %d)",
+ suspect_start, suspect_end);
+ if (actual_crc != expected_crc) {
+ GST_DEBUG_OBJECT (flacparse, "checksum did not match");
continue;
+ }
}
*ret = i;
flacparse->block_size = block_size;
flacparse->offset = GST_BUFFER_OFFSET (buffer);
ret =
gst_flac_parse_frame_header_is_valid (flacparse,
- GST_BUFFER_DATA (buffer), GST_BUFFER_SIZE (buffer), TRUE, NULL);
+ GST_BUFFER_DATA (buffer), GST_BUFFER_SIZE (buffer), TRUE, NULL, NULL);
if (ret != FRAME_HEADER_VALID) {
GST_ERROR_OBJECT (flacparse,
"Baseclass didn't provide a complete frame");