On 03/03/16 00:52, Vittorio Giovara wrote:
> From: Katerina Barone-Adesi <kateri...@gmail.com>
> 
> The original code left-shifts negative values, which is undefined
> in the C99 specification (the one used during normal Libav compilation).
> This change multiplies by (1 << shift), which is functionally equivalent,
> but has defined behavior.
> 
> With this change, fate-idct8x8 compiled with --fsanitize=undefined works.
> 
> Bug-Id: 686
> ---
>  libavcodec/jfdctint_template.c    |  4 ++--
>  libavcodec/jrevdct.c              | 36 ++++++++++++++++++------------------
>  libavcodec/simple_idct_template.c | 10 +++++-----
>  3 files changed, 25 insertions(+), 25 deletions(-)

Seems fine.
_______________________________________________
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to