diff mbox

[FFmpeg-devel] avcodec/mjpegbdec: Fix yuv444 pix_fmt detection

Message ID 20191120112246.54562-1-alex@mogurenko.com
State Accepted
Commit b6571777d28d59308e622883c9fe7d0765e406a8
Headers show

Commit Message

Alex Mogurenko Nov. 20, 2019, 11:22 a.m. UTC 
by default adobe_transform set to 0 and because of that mjpegb decoder detects yuv444 pix fmt as bgrp
---
 libavcodec/mjpegbdec.c | 1 +
 1 file changed, 1 insertion(+)

Comments

Carl Eugen Hoyos Nov. 20, 2019, 12:57 p.m. UTC | #1 
> Am 20.11.2019 um 12:22 schrieb Alex Mogurenko <alex@mogurenko.com>:
> 
> by default adobe_transform set to 0 and because of that mjpegb decoder detects yuv444 pix fmt as bgrp

Please provide a sample that gets fixed by this patch.

Carl Eugen
Alex Mogurenko Nov. 20, 2019, 1:41 p.m. UTC | #2 
On Wed, Nov 20, 2019 at 2:57 PM Carl Eugen Hoyos <ceffmpeg@gmail.com> wrote:

>
>
> > Am 20.11.2019 um 12:22 schrieb Alex Mogurenko <alex@mogurenko.com>:
> >
> > by default adobe_transform set to 0 and because of that mjpegb decoder
> detects yuv444 pix fmt as bgrp
>
> Please provide a sample that gets fixed by this patch.
>
>
Where is preferable to upload it?
Carl Eugen Hoyos Nov. 20, 2019, 1:57 p.m. UTC | #3 
> Am 20.11.2019 um 14:41 schrieb Alex Mogurenko <alex@mogurenko.com>:
> 
>> On Wed, Nov 20, 2019 at 2:57 PM Carl Eugen Hoyos <ceffmpeg@gmail.com> wrote:
>> 
>> 
>> 
>>> Am 20.11.2019 um 12:22 schrieb Alex Mogurenko <alex@mogurenko.com>:
>>> 
>>> by default adobe_transform set to 0 and because of that mjpegb decoder
>> detects yuv444 pix fmt as bgrp
>> 
>> Please provide a sample that gets fixed by this patch.
>> 
>> 
> Where is preferable to upload it?

Any file hoster that does not require registration.

Carl Eugen
Alex Mogurenko Nov. 20, 2019, 2:20 p.m. UTC | #4 
On Wed, Nov 20, 2019 at 3:57 PM Carl Eugen Hoyos <ceffmpeg@gmail.com> wrote:

> > Am 20.11.2019 um 14:41 schrieb Alex Mogurenko <alex@mogurenko.com>:
> >
> >> On Wed, Nov 20, 2019 at 2:57 PM Carl Eugen Hoyos <ceffmpeg@gmail.com>
> wrote:
> >>
> >>
> >>
> >>> Am 20.11.2019 um 12:22 schrieb Alex Mogurenko <alex@mogurenko.com>:
> >>>
> >>> by default adobe_transform set to 0 and because of that mjpegb decoder
> >> detects yuv444 pix fmt as bgrp
> >>
> >> Please provide a sample that gets fixed by this patch.
> >>
> >>
> > Where is preferable to upload it?
>
> Any file hoster that does not require registration.
>

https://send.firefox.com/download/12d9f70cb957832d/#JAWkB77yZQ0OxM8x95fOvg
Carl Eugen Hoyos Nov. 20, 2019, 5:36 p.m. UTC | #5 
Am Mi., 20. Nov. 2019 um 12:29 Uhr schrieb Alex Mogurenko <alex@mogurenko.com>:
>
> by default adobe_transform set to 0 and because of that mjpegb decoder detects yuv444 pix fmt as bgrp
> ---
>  libavcodec/mjpegbdec.c | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/libavcodec/mjpegbdec.c b/libavcodec/mjpegbdec.c
> index 37d7bb8228..70ff4cf563 100644
> --- a/libavcodec/mjpegbdec.c
> +++ b/libavcodec/mjpegbdec.c
> @@ -56,6 +56,7 @@ static int mjpegb_decode_frame(AVCodecContext *avctx,
>      buf_ptr = buf;
>      buf_end = buf + buf_size;
>      s->got_picture = 0;
> +    s->adobe_transform = -1;
>
>  read_header:
>      /* reset on every SOI */

Patch applied.

Thank you, Carl Eugen
diff mbox

Patch

diff --git a/libavcodec/mjpegbdec.c b/libavcodec/mjpegbdec.c
index 37d7bb8228..70ff4cf563 100644
--- a/libavcodec/mjpegbdec.c
+++ b/libavcodec/mjpegbdec.c
@@ -56,6 +56,7 @@  static int mjpegb_decode_frame(AVCodecContext *avctx,
     buf_ptr = buf;
     buf_end = buf + buf_size;
     s->got_picture = 0;
+    s->adobe_transform = -1;
 
 read_header:
     /* reset on every SOI */