summaryrefslogtreecommitdiffstats
path: root/arch/arm/lib
diff options
context:
space:
mode:
authorRussell King <rmk@dyn-67.arm.linux.org.uk>2005-11-10 11:40:53 +0000
committerRussell King <rmk+kernel@arm.linux.org.uk>2005-11-10 11:40:53 +0000
commitaf36bef0c5bb82f361ebb2f106f11d0f63dac887 (patch)
treec8fcea1482c15fefc07044e2ff2a77a54310fb19 /arch/arm/lib
parent3b44f137b9a846c5452d9e6e1271b79b1dbcc942 (diff)
downloadop-kernel-dev-af36bef0c5bb82f361ebb2f106f11d0f63dac887.zip
op-kernel-dev-af36bef0c5bb82f361ebb2f106f11d0f63dac887.tar.gz
[ARM] Fix csumpartial corner case
Ji-In Park discovered a bug in csumpartial which caused wrong checksums with misaligned buffers. Signed-off-by: Russell King <rmk+kernel@arm.linux.org.uk>
Diffstat (limited to 'arch/arm/lib')
-rw-r--r--arch/arm/lib/csumpartial.S4
1 files changed, 4 insertions, 0 deletions
diff --git a/arch/arm/lib/csumpartial.S b/arch/arm/lib/csumpartial.S
index cb5e370..fe797cf 100644
--- a/arch/arm/lib/csumpartial.S
+++ b/arch/arm/lib/csumpartial.S
@@ -39,6 +39,7 @@ td3 .req lr
/* we must have at least one byte. */
tst buf, #1 @ odd address?
+ movne sum, sum, ror #8
ldrneb td0, [buf], #1
subne len, len, #1
adcnes sum, sum, td0, put_byte_1
@@ -103,6 +104,9 @@ ENTRY(csum_partial)
cmp len, #8 @ Ensure that we have at least
blo .less8 @ 8 bytes to copy.
+ tst buf, #1
+ movne sum, sum, ror #8
+
adds sum, sum, #0 @ C = 0
tst buf, #3 @ Test destination alignment
blne .not_aligned @ aligh destination, return here
OpenPOWER on IntegriCloud