[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH] hw/display/exynos4210_fimd: Fix bit-swapping code
From: |
Peter Maydell |
Subject: |
[Qemu-devel] [PATCH] hw/display/exynos4210_fimd: Fix bit-swapping code |
Date: |
Fri, 29 May 2015 16:16:55 +0100 |
fimd_swap_data() includes code to reverse the bits in a
64-bit integer, but an off-by-one error meant that it would
try to shift off the top of the integer. Correct the bug
(spotted by Coverity).
Signed-off-by: Peter Maydell <address@hidden>
---
Compile-tested only, I have no exynos images to test with.
hw/display/exynos4210_fimd.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/hw/display/exynos4210_fimd.c b/hw/display/exynos4210_fimd.c
index 45c62af..3e59ee4 100644
--- a/hw/display/exynos4210_fimd.c
+++ b/hw/display/exynos4210_fimd.c
@@ -337,7 +337,7 @@ static inline void fimd_swap_data(unsigned int swap_ctl,
uint64_t *data)
if (swap_ctl & FIMD_WINCON_SWAP_BITS) {
res = 0;
for (i = 0; i < 64; i++) {
- if (x & (1ULL << (64 - i))) {
+ if (x & (1ULL << (63 - i))) {
res |= (1ULL << i);
}
}
--
1.9.1
- [Qemu-devel] [PATCH] hw/display/exynos4210_fimd: Fix bit-swapping code,
Peter Maydell <=