Re: [libav-devel] [PATCH 3/3] matroskadec: allow flavor 0 files to contain RealAudio streams
Hi, On Nov 2, 2017 07:30, "Martin Storsjö" wrote: On Thu, 2 Nov 2017, Sean McGovern wrote: Hi, > > On Nov 2, 2017 02:50, "Martin Storsjö" wrote: > > On Wed, 1 Nov 2017, Sean McGovern wrote: > > Regression since 569d18aa9dc989c37bb4d4b968026fe5afa6fff9. > >> >> Bug-Id: 1055 >> Cc: libav-sta...@libav.org >> --- >> libavformat/matroskadec.c | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/libavformat/matroskadec.c b/libavformat/matroskadec.c >> index 9c523fb..bad750c 100644 >> --- a/libavformat/matroskadec.c >> +++ b/libavformat/matroskadec.c >> @@ -1919,7 +1919,7 @@ static int matroska_parse_tracks(AVFormatContext >> *s) >> track->audio.sub_packet_h= avio_rb16(&b); >> track->audio.frame_size = avio_rb16(&b); >> track->audio.sub_packet_size = avio_rb16(&b); >> -if (flavor <= 0 || >> +if (flavor < 0 || >> track->audio.coded_framesize <= 0 || >> track->audio.sub_packet_h<= 0 || >> track->audio.frame_size <= 0 || >> -- >> 2.7.4 >> >> > Possibly ok (the commit message of the offending commit doesn't indicate > which sample it was about so I can't easily check whether that sample > needed to check for strictly flavor <= 0 or not). > > // Martin > ___ > libav-devel mailing list > libav-devel@libav.org > https://lists.libav.org/mailman/listinfo/libav-devel > > > I was probably half-asleep when I wrote the commit message for this. It > should really be something like: > > matroskadec: allow RealAudio/Cook/Sipro streams of flavor 0 > > ...Or something similar (suggestions?) > > The sample used to fix this issue is from BZ #268 [1]. > What I meant was; the original commit, 569d18aa9dc989c37bb4d4b968026fe5afa6fff9, tried to fix some crash with some fuzzed input files. We should ideally make sure those crashes stay fixed. Since I didn't write out which sample it was about, it's not very easy to find out now which one it might have been though, to verify we don't reopen that crash bug. // Martin ___ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listinfo/libav-devel I hope this is still OK -- the other sanity checks in that if clause are left untouched. Thanks, -- Sean McGovern ___ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listinfo/libav-devel
Re: [libav-devel] [PATCH 3/3] matroskadec: allow flavor 0 files to contain RealAudio streams
On Thu, 2 Nov 2017, Sean McGovern wrote: Hi, On Nov 2, 2017 02:50, "Martin Storsjö" wrote: On Wed, 1 Nov 2017, Sean McGovern wrote: Regression since 569d18aa9dc989c37bb4d4b968026fe5afa6fff9. Bug-Id: 1055 Cc: libav-sta...@libav.org --- libavformat/matroskadec.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/matroskadec.c b/libavformat/matroskadec.c index 9c523fb..bad750c 100644 --- a/libavformat/matroskadec.c +++ b/libavformat/matroskadec.c @@ -1919,7 +1919,7 @@ static int matroska_parse_tracks(AVFormatContext *s) track->audio.sub_packet_h= avio_rb16(&b); track->audio.frame_size = avio_rb16(&b); track->audio.sub_packet_size = avio_rb16(&b); -if (flavor <= 0 || +if (flavor < 0 || track->audio.coded_framesize <= 0 || track->audio.sub_packet_h<= 0 || track->audio.frame_size <= 0 || -- 2.7.4 Possibly ok (the commit message of the offending commit doesn't indicate which sample it was about so I can't easily check whether that sample needed to check for strictly flavor <= 0 or not). // Martin ___ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listinfo/libav-devel I was probably half-asleep when I wrote the commit message for this. It should really be something like: matroskadec: allow RealAudio/Cook/Sipro streams of flavor 0 ...Or something similar (suggestions?) The sample used to fix this issue is from BZ #268 [1]. What I meant was; the original commit, 569d18aa9dc989c37bb4d4b968026fe5afa6fff9, tried to fix some crash with some fuzzed input files. We should ideally make sure those crashes stay fixed. Since I didn't write out which sample it was about, it's not very easy to find out now which one it might have been though, to verify we don't reopen that crash bug. // Martin ___ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listinfo/libav-devel
Re: [libav-devel] [PATCH 3/3] matroskadec: allow flavor 0 files to contain RealAudio streams
Hi, On Nov 2, 2017 02:50, "Martin Storsjö" wrote: On Wed, 1 Nov 2017, Sean McGovern wrote: Regression since 569d18aa9dc989c37bb4d4b968026fe5afa6fff9. > > Bug-Id: 1055 > Cc: libav-sta...@libav.org > --- > libavformat/matroskadec.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavformat/matroskadec.c b/libavformat/matroskadec.c > index 9c523fb..bad750c 100644 > --- a/libavformat/matroskadec.c > +++ b/libavformat/matroskadec.c > @@ -1919,7 +1919,7 @@ static int matroska_parse_tracks(AVFormatContext *s) > track->audio.sub_packet_h= avio_rb16(&b); > track->audio.frame_size = avio_rb16(&b); > track->audio.sub_packet_size = avio_rb16(&b); > -if (flavor <= 0 || > +if (flavor < 0 || > track->audio.coded_framesize <= 0 || > track->audio.sub_packet_h<= 0 || > track->audio.frame_size <= 0 || > -- > 2.7.4 > Possibly ok (the commit message of the offending commit doesn't indicate which sample it was about so I can't easily check whether that sample needed to check for strictly flavor <= 0 or not). // Martin ___ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listinfo/libav-devel I was probably half-asleep when I wrote the commit message for this. It should really be something like: matroskadec: allow RealAudio/Cook/Sipro streams of flavor 0 ...Or something similar (suggestions?) The sample used to fix this issue is from BZ #268 [1]. -- Sean McG. [1] https://bugzilla.libav.org/show_bug.cgi?id=268 ___ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listinfo/libav-devel
Re: [libav-devel] [PATCH 3/3] matroskadec: allow flavor 0 files to contain RealAudio streams
On Wed, 1 Nov 2017, Sean McGovern wrote: Regression since 569d18aa9dc989c37bb4d4b968026fe5afa6fff9. Bug-Id: 1055 Cc: libav-sta...@libav.org --- libavformat/matroskadec.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/matroskadec.c b/libavformat/matroskadec.c index 9c523fb..bad750c 100644 --- a/libavformat/matroskadec.c +++ b/libavformat/matroskadec.c @@ -1919,7 +1919,7 @@ static int matroska_parse_tracks(AVFormatContext *s) track->audio.sub_packet_h= avio_rb16(&b); track->audio.frame_size = avio_rb16(&b); track->audio.sub_packet_size = avio_rb16(&b); -if (flavor <= 0 || +if (flavor < 0 || track->audio.coded_framesize <= 0 || track->audio.sub_packet_h<= 0 || track->audio.frame_size <= 0 || -- 2.7.4 Possibly ok (the commit message of the offending commit doesn't indicate which sample it was about so I can't easily check whether that sample needed to check for strictly flavor <= 0 or not). // Martin ___ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listinfo/libav-devel
[libav-devel] [PATCH 3/3] matroskadec: allow flavor 0 files to contain RealAudio streams
Regression since 569d18aa9dc989c37bb4d4b968026fe5afa6fff9. Bug-Id: 1055 Cc: libav-sta...@libav.org --- libavformat/matroskadec.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/matroskadec.c b/libavformat/matroskadec.c index 9c523fb..bad750c 100644 --- a/libavformat/matroskadec.c +++ b/libavformat/matroskadec.c @@ -1919,7 +1919,7 @@ static int matroska_parse_tracks(AVFormatContext *s) track->audio.sub_packet_h= avio_rb16(&b); track->audio.frame_size = avio_rb16(&b); track->audio.sub_packet_size = avio_rb16(&b); -if (flavor <= 0 || +if (flavor < 0 || track->audio.coded_framesize <= 0 || track->audio.sub_packet_h<= 0 || track->audio.frame_size <= 0 || -- 2.7.4 ___ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listinfo/libav-devel