[FFmpeg-devel] [PATCH v2 2/2] avformat/mvdec: re-indent after last commit

2021-12-31 Thread John-Paul Stewart
Signed-off-by: John-Paul Stewart --- libavformat/mvdec.c | 72 ++--- 1 file changed, 36 insertions(+), 36 deletions(-) diff --git a/libavformat/mvdec.c b/libavformat/mvdec.c index 0f0474141b..4f233aff5f 100644 --- a/libavformat/mvdec.c +++ b/libavformat

[FFmpeg-devel] [PATCH v2 1/2] avformat/mvdec: make audio stream conditional

2021-12-31 Thread John-Paul Stewart
samples in it. There is a flag value in the format to indicate whether or not audio is present. This patch checks that and behaves accordingly. Signed-off-by: John-Paul Stewart --- libavformat/mvdec.c | 22 +++--- 1 file changed, 19 insertions(+), 3 deletions(-) diff --git a

[FFmpeg-devel] [PATCH v2 0/2] avformat/mvdec: make audio stream conditional

2021-12-31 Thread John-Paul Stewart
Changed in v2: Allocate the audio stream first to maintain consistent behaviour with prior code. Recent discussion on the list led me to realize that libavformat was unconditionally creating an audio stream for all SGI movie format (version 2) files, even when no audio is present in the fi

Re: [FFmpeg-devel] [PATCH 0/2] avformat/mvdec: make audio stream conditional

2021-12-31 Thread John-Paul Stewart
On 2021-12-31 19:33, Andreas Rheinhardt wrote: > John-Paul Stewart: >> On 2021-12-31 17:19, Andreas Rheinhardt wrote: >>> John-Paul Stewart: >>>> Recent discussion on the list led me to realize that libavformat was >>>> unconditionally creatin

Re: [FFmpeg-devel] [PATCH 0/2] avformat/mvdec: make audio stream conditional

2021-12-31 Thread John-Paul Stewart
On 2021-12-31 17:19, Andreas Rheinhardt wrote: > John-Paul Stewart: >> Recent discussion on the list led me to realize that libavformat was >> unconditionally creating an audio stream for all SGI movie format >> (version 2) files, even when no audio is present in the file.

[FFmpeg-devel] [PATCH 2/2] avformat/mvdec: re-indent after last commit

2021-12-31 Thread John-Paul Stewart
Signed-off-by: John-Paul Stewart --- libavformat/mvdec.c | 64 ++--- 1 file changed, 32 insertions(+), 32 deletions(-) diff --git a/libavformat/mvdec.c b/libavformat/mvdec.c index 7493bcc762..469ac32092 100644 --- a/libavformat/mvdec.c +++ b/libavformat

[FFmpeg-devel] [PATCH 1/2] avformat/mvdec: make audio stream conditional

2021-12-31 Thread John-Paul Stewart
samples in it. There is a flag value in the format to indicate whether or not audio is present. This patch checks that and behaves accordingly. Signed-off-by: John-Paul Stewart --- libavformat/mvdec.c | 33 - 1 file changed, 24 insertions(+), 9 deletions(-) diff

[FFmpeg-devel] [PATCH 0/2] avformat/mvdec: make audio stream conditional

2021-12-31 Thread John-Paul Stewart
Recent discussion on the list led me to realize that libavformat was unconditionally creating an audio stream for all SGI movie format (version 2) files, even when no audio is present in the file. A sample of a movie file with no audio can be found at http://www.personalprojects.net/ffmpeg/s

Re: [FFmpeg-devel] [PATCH 5/5] avformat/mvdec: Check bytes_per_sample

2021-12-24 Thread John-Paul Stewart
On 2021-12-24 11:58, Michael Niedermayer wrote: > On Thu, Dec 23, 2021 at 10:32:12PM -0500, John-Paul Stewart wrote: >> On 2021-12-23 16:15, Michael Niedermayer wrote: >>> Fixes: division by zero >>> Fixes: >>> 42814/clusterfuzz-testcase-minimized-f