[FFmpeg-cvslog] avcodec/dca: fix sync word search error condition
foo86
git at videolan.org
Mon May 2 23:59:51 CEST 2016
ffmpeg | branch: master | foo86 <foobaz86 at gmail.com> | Mon May 2 23:44:47 2016 +0300| [ce2f9fdb0a92956aedfa2c564d1374a2f1eebfbd] | committer: Michael Niedermayer
avcodec/dca: fix sync word search error condition
This didn't actually check if sync word was found and always errored out
with "-err_detect explode" option enabled.
Signed-off-by: Michael Niedermayer <michael at niedermayer.cc>
> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=ce2f9fdb0a92956aedfa2c564d1374a2f1eebfbd
---
libavcodec/dca_core.c | 15 +++++++++------
1 file changed, 9 insertions(+), 6 deletions(-)
diff --git a/libavcodec/dca_core.c b/libavcodec/dca_core.c
index fc4fdf4..58c78f7 100644
--- a/libavcodec/dca_core.c
+++ b/libavcodec/dca_core.c
@@ -1844,9 +1844,10 @@ static int parse_optional_info(DCACoreDecoder *s)
}
}
- if (s->avctx->err_recognition & AV_EF_EXPLODE) {
+ if (!s->xch_pos) {
av_log(s->avctx, AV_LOG_ERROR, "XCH sync word not found\n");
- return AVERROR_INVALIDDATA;
+ if (s->avctx->err_recognition & AV_EF_EXPLODE)
+ return AVERROR_INVALIDDATA;
}
break;
@@ -1866,9 +1867,10 @@ static int parse_optional_info(DCACoreDecoder *s)
}
}
- if (s->avctx->err_recognition & AV_EF_EXPLODE) {
+ if (!s->x96_pos) {
av_log(s->avctx, AV_LOG_ERROR, "X96 sync word not found\n");
- return AVERROR_INVALIDDATA;
+ if (s->avctx->err_recognition & AV_EF_EXPLODE)
+ return AVERROR_INVALIDDATA;
}
break;
@@ -1891,9 +1893,10 @@ static int parse_optional_info(DCACoreDecoder *s)
}
}
- if (s->avctx->err_recognition & AV_EF_EXPLODE) {
+ if (!s->xxch_pos) {
av_log(s->avctx, AV_LOG_ERROR, "XXCH sync word not found\n");
- return AVERROR_INVALIDDATA;
+ if (s->avctx->err_recognition & AV_EF_EXPLODE)
+ return AVERROR_INVALIDDATA;
}
break;
}
More information about the ffmpeg-cvslog
mailing list