diff options
| author | Loren Merritt <lorenm@u.washington.edu> | 2012-09-22 03:43:16 +0400 |
|---|---|---|
| committer | Michael Niedermayer <michaelni@gmx.at> | 2012-09-22 05:17:28 +0400 |
| commit | 1b1b902e2ce19e0276bf847cc87384ec43d230e3 (patch) | |
| tree | 2ab4d75f9201f5746a66e132929e52e05d985adf /libavfilter/vf_hqdn3d.c | |
| parent | ea5bd7ea6f3b817bdee6355a2b93d512d2ab6bf2 (diff) | |
hqdn3d: Fix out of array read in LOWPASS
Fixes ticket1752
Commit message by commiter
Signed-off-by: Michael Niedermayer <michaelni@gmx.at>
Diffstat (limited to 'libavfilter/vf_hqdn3d.c')
| -rw-r--r-- | libavfilter/vf_hqdn3d.c | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/libavfilter/vf_hqdn3d.c b/libavfilter/vf_hqdn3d.c index 51c7e879ab..2fe70563a1 100644 --- a/libavfilter/vf_hqdn3d.c +++ b/libavfilter/vf_hqdn3d.c @@ -50,10 +50,9 @@ void ff_hqdn3d_row_10_x86(uint8_t *src, uint8_t *dst, uint16_t *line_ant, uint16 void ff_hqdn3d_row_16_x86(uint8_t *src, uint8_t *dst, uint16_t *line_ant, uint16_t *frame_ant, ptrdiff_t w, int16_t *spatial, int16_t *temporal); #define LUT_BITS (depth==16 ? 8 : 4) -#define RIGHTSHIFT(a,b) (((a)+(((1<<(b))-1)>>1))>>(b)) -#define LOAD(x) ((depth==8 ? src[x] : AV_RN16A(src+(x)*2)) << (16-depth)) -#define STORE(x,val) (depth==8 ? dst[x] = RIGHTSHIFT(val, 16-depth)\ - : AV_WN16A(dst+(x)*2, RIGHTSHIFT(val, 16-depth))) +#define LOAD(x) (((depth==8 ? src[x] : AV_RN16A(src+(x)*2)) << (16-depth)) + (((1<<(16-depth))-1)>>1)) +#define STORE(x,val) (depth==8 ? dst[x] = (val) >> (16-depth)\ + : AV_WN16A(dst+(x)*2, (val) >> (16-depth))) av_always_inline static uint32_t lowpass(int prev, int cur, int16_t *coef, int depth) |