ARM: Fix csum_partial_copy_from_user()
Using the parent functions frame pointer to access our arguments is completely wrong, whether or not we're building with frame pointers or not. What we should be using is the stack pointer to get at the word above the registers we stacked ourselves. Reported-by: Bosko Radivojevic <bosko.radivojevic@gmail.com> Tested-by: Bosko Radivojevic <bosko.radivojevic@gmail.com> Signed-off-by: Russell King <rmk+kernel@arm.linux.org.uk>
This commit is contained in:
parent
f9578fc078
commit
4609a179c9
|
@ -71,7 +71,7 @@
|
||||||
.pushsection .fixup,"ax"
|
.pushsection .fixup,"ax"
|
||||||
.align 4
|
.align 4
|
||||||
9001: mov r4, #-EFAULT
|
9001: mov r4, #-EFAULT
|
||||||
ldr r5, [fp, #4] @ *err_ptr
|
ldr r5, [sp, #8*4] @ *err_ptr
|
||||||
str r4, [r5]
|
str r4, [r5]
|
||||||
ldmia sp, {r1, r2} @ retrieve dst, len
|
ldmia sp, {r1, r2} @ retrieve dst, len
|
||||||
add r2, r2, r1
|
add r2, r2, r1
|
||||||
|
|
Loading…
Reference in New Issue