On Thu, Nov 17, 2016 at 10:52:30PM +0100, Andreas Cadhalpun wrote:
> This fixes division by zero crashes.
>
> Signed-off-by: Andreas Cadhalpun <andreas.cadhal...@googlemail.com>
> ---
> libavformat/rmdec.c | 4 ++++
> 1 file changed, 4 insertions(+)
>
> diff --git a/libavformat/rmdec.c b/libavformat/rmdec.c
> index d175862..4d56529 100644
> --- a/libavformat/rmdec.c
> +++ b/libavformat/rmdec.c
> @@ -934,6 +934,10 @@ ff_rm_parse_packet (AVFormatContext *s, AVIOContext *pb,
>
> ast->sub_packet_cnt = 0;
> rm->audio_stream_num = st->index;
> + if (st->codecpar->block_align <= 0) {
> + av_log(s, AV_LOG_ERROR, "Invalid block alignment %d\n",
> st->codecpar->block_align);
> + return AVERROR_INVALIDDATA;
> + }
> rm->audio_pkt_cnt = h * w / st->codecpar->block_align;indention looks odd also i think this is already checked in rm_read_audio_stream_info() is that check somehow broken ? [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Does the universe only have a finite lifespan? No, its going to go on forever, its just that you wont like living in it. -- Hiranya Peiri
signature.asc
Description: Digital signature
_______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
