[FFmpeg-cvslog] Fix a regression in the dca parser introduced by dts express support.
Hendrik Leppkes
git at videolan.org
Fri Oct 28 15:33:32 CEST 2011
ffmpeg | branch: master | Hendrik Leppkes <h.leppkes at gmail.com> | Mon Jul 25 21:32:53 2011 +0200| [022d22e5810d1c90f618fddd751b1f03502d6021] | committer: Michael Niedermayer
Fix a regression in the dca parser introduced by dts express support.
The dca parser needs to check that the framesize is actually of a complete
frame, and not of a partial DTS-HD frame, which is not constant size, and
thus the check would fail.
(cherry picked from commit ebc0ccb9af59b78732e82c09f8c90b1d46b478e0)
Review-by: Benjamin Larsson
Signed-off-by: Michael Niedermayer <michaelni at gmx.at>
> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=022d22e5810d1c90f618fddd751b1f03502d6021
---
libavcodec/dca_parser.c | 3 ++-
1 files changed, 2 insertions(+), 1 deletions(-)
diff --git a/libavcodec/dca_parser.c b/libavcodec/dca_parser.c
index ff91769..74e6522 100644
--- a/libavcodec/dca_parser.c
+++ b/libavcodec/dca_parser.c
@@ -74,7 +74,8 @@ static int dca_find_frame_end(DCAParseContext * pc1, const uint8_t * buf,
if (IS_MARKER(state, i, buf, buf_size) && (state == pc1->lastmarker || pc1->lastmarker == DCA_HD_MARKER)) {
if(pc1->framesize > pc1->size)
continue;
- if(!pc1->framesize){
+ // We have to check that we really read a full frame here, and that it isn't a pure HD frame, because their size is not constant.
+ if(!pc1->framesize && state == pc1->lastmarker && state != DCA_HD_MARKER){
pc1->framesize = pc1->hd_pos ? pc1->hd_pos : pc1->size;
}
pc->frame_start_found = 0;
More information about the ffmpeg-cvslog
mailing list