diff options
| author | Yuli Wang <wangyuli@uniontech.com> | 2025-02-13 12:02:40 +0800 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2025-02-21 14:10:46 +0100 |
| commit | d6508ffff32b44b6d0de06704034e4eef1c307a7 (patch) | |
| tree | 235f2b7fe5e3dfd15540596503150abd7711c111 /arch/loongarch | |
| parent | 70779eeb69253549e78510209826ea44c17ac4e6 (diff) | |
| download | linux-d6508ffff32b44b6d0de06704034e4eef1c307a7.tar.gz linux-d6508ffff32b44b6d0de06704034e4eef1c307a7.tar.bz2 linux-d6508ffff32b44b6d0de06704034e4eef1c307a7.zip | |
LoongArch: csum: Fix OoB access in IP checksum code for negative lengths
[ Upstream commit 6287f1a8c16138c2ec750953e35039634018c84a ]
Commit 69e3a6aa6be2 ("LoongArch: Add checksum optimization for 64-bit
system") would cause an undefined shift and an out-of-bounds read.
Commit 8bd795fedb84 ("arm64: csum: Fix OoB access in IP checksum code
for negative lengths") fixes the same issue on ARM64.
Fixes: 69e3a6aa6be2 ("LoongArch: Add checksum optimization for 64-bit system")
Co-developed-by: Wentao Guan <guanwentao@uniontech.com>
Signed-off-by: Wentao Guan <guanwentao@uniontech.com>
Signed-off-by: Yuli Wang <wangyuli@uniontech.com>
Signed-off-by: Huacai Chen <chenhuacai@loongson.cn>
Signed-off-by: Sasha Levin <sashal@kernel.org>
Diffstat (limited to 'arch/loongarch')
| -rw-r--r-- | arch/loongarch/lib/csum.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/arch/loongarch/lib/csum.c b/arch/loongarch/lib/csum.c index a5e84b403c3b..df309ae4045d 100644 --- a/arch/loongarch/lib/csum.c +++ b/arch/loongarch/lib/csum.c @@ -25,7 +25,7 @@ unsigned int __no_sanitize_address do_csum(const unsigned char *buff, int len) const u64 *ptr; u64 data, sum64 = 0; - if (unlikely(len == 0)) + if (unlikely(len <= 0)) return 0; offset = (unsigned long)buff & 7; |