On Sun, Oct 04, 2015 at 09:21:55PM -0400, Ganesh Ajjanagadde wrote:
> Fixes CID 1322359, CID 1322358.
> 
> Signed-off-by: Ganesh Ajjanagadde <gajjanaga...@gmail.com>
> ---
>  libavcodec/pngdec.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/libavcodec/pngdec.c b/libavcodec/pngdec.c
> index d180141..fe22225 100644
> --- a/libavcodec/pngdec.c
> +++ b/libavcodec/pngdec.c
> @@ -1006,7 +1006,7 @@ static int handle_p_frame_apng(AVCodecContext *avctx, 
> PNGDecContext *s,
>              uint8_t *background = buffer + s->image_linesize * y + s->bpp * 
> s->x_offset;
>              for (x = s->x_offset; x < s->x_offset + s->cur_w; ++x, 
> foreground += s->bpp, background += s->bpp) {
>                  size_t b;
> -                uint8_t foreground_alpha, background_alpha, output_alpha;
> +                uint8_t foreground_alpha = 0, background_alpha, output_alpha;

this looks very odd
have you checked that the reference png implementation and png spec
set foreground_alpha to 0 for cases that FFmpeg doesnt implement ? ;)

warnings can point to bugs and if so the bug should be fixed which
should make the warning disappear. If a warning doesnt point to a bug
then it should be silenced in some clean way

It seems you try to silence this one without really realizing
what is wrong with the codepath in which this warning occurs

I didnt write it but to me it seems the if() a few lines above is
intended to stop this codepath
see 'avpriv_request_sample(avctx, "Blending with pixel format %s",'

it doesnt work as blend_op is assumed to have only 2 values but
this is not checked for anywhere.
blend_op should be checked when/before it is set


[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

I do not agree with what you have to say, but I'll defend to the death your
right to say it. -- Voltaire

Attachment: signature.asc
Description: Digital signature

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Reply via email to