Browse Source

avfilter/vf_paletteuse: Fix left shift outside of range of int

by keeping the variable uint32_t which in this situation is the natural
type anyway. This affected the FATE-test filter-paletteuse-sierra2_4a.

Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@gmail.com>
master
Andreas Rheinhardt Andreas Rheinhardt 6 years ago
parent
commit
797c2ecc8f
1 changed files with 3 additions and 2 deletions
  1. +3
    -2
      libavfilter/vf_paletteuse.c

+ 3
- 2
libavfilter/vf_paletteuse.c View File

@@ -152,9 +152,10 @@ static int query_formats(AVFilterContext *ctx)
return 0;
}

static av_always_inline int dither_color(uint32_t px, int er, int eg, int eb, int scale, int shift)
static av_always_inline uint32_t dither_color(uint32_t px, int er, int eg,
int eb, int scale, int shift)
{
return av_clip_uint8( px >> 24 ) << 24
return px >> 24 << 24
| av_clip_uint8((px >> 16 & 0xff) + ((er * scale) / (1<<shift))) << 16
| av_clip_uint8((px >> 8 & 0xff) + ((eg * scale) / (1<<shift))) << 8
| av_clip_uint8((px & 0xff) + ((eb * scale) / (1<<shift)));


Loading…
Cancel
Save