har0ke/FFmpeg
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

dsputil_mmx: fix reading prior of the src array in sub_hfyu_median_prediction()

Browse Source 
This should fix the utvideoenc valgrind failure

Signed-off-by: Michael Niedermayer <michaelni@gmx.at>
This commit is contained in:
Michael Niedermayer 2012-09-27 14:06:54 +00:00 committed by Derek Buitenhuis
parent 97cfa55eea
commit 791b5954bc
1 changed files with 3 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
libavcodec/x86/dsputilenc_mmx.c
View File

@ -849,8 +849,9 @@ static void sub_hfyu_median_prediction_mmx2(uint8_t *dst, const uint8_t *src1, c
uint8_t l, lt; uint8_t l, lt;
__asm__ volatile( __asm__ volatile(
"movq (%1, %0), %%mm0 \n\t" // LT
"psllq $8, %%mm0 \n\t"
"1: \n\t" "1: \n\t"
"movq -1(%1, %0), %%mm0 \n\t" // LT
"movq (%1, %0), %%mm1 \n\t" // T "movq (%1, %0), %%mm1 \n\t" // T
"movq -1(%2, %0), %%mm2 \n\t" // L "movq -1(%2, %0), %%mm2 \n\t" // L
"movq (%2, %0), %%mm3 \n\t" // X "movq (%2, %0), %%mm3 \n\t" // X
@ -865,6 +866,7 @@ static void sub_hfyu_median_prediction_mmx2(uint8_t *dst, const uint8_t *src1, c
"psubb %%mm4, %%mm3 \n\t" // dst - pred "psubb %%mm4, %%mm3 \n\t" // dst - pred
"movq %%mm3, (%3, %0) \n\t" "movq %%mm3, (%3, %0) \n\t"
"add $8, %0 \n\t" "add $8, %0 \n\t"
"movq -1(%1, %0), %%mm0 \n\t" // LT
"cmp %4, %0 \n\t" "cmp %4, %0 \n\t"
" jb 1b \n\t" " jb 1b \n\t"
: "+r" (i) : "+r" (i)