[PATCH] fbcon: Console Rotation - Fix wrong shift calculation
authorAntonino A. Daplas <adaplas@gmail.com>
Tue, 22 Nov 2005 05:32:25 +0000 (21:32 -0800)
committerLinus Torvalds <torvalds@g5.osdl.org>
Tue, 22 Nov 2005 17:13:43 +0000 (09:13 -0800)
The shift value (amount to shift the bitmap so first pixel starts at
origin(0,0)) is incorrect.  This causes corrupted characters or a kernel crash
if fontwidth is not divisible by 8 at 270 degrees, or fontheight not divisible
by 8 at 180 degrees.

Report and part of the fix contributed by Knut Petersen.

Signed-off-by: Antonino Daplas <adaplas@pol.net>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
drivers/video/console/fbcon_rotate.h

index 90c672096c2e3e4e2340771a55930350b3521b03..e504fbf5c6045bedb1319d76f968b0c4e425007f 100644 (file)
@@ -49,7 +49,7 @@ static inline void pattern_set_bit(u32 x, u32 y, u32 pitch, char *pat)
 static inline void rotate_ud(const char *in, char *out, u32 width, u32 height)
 {
        int i, j;
-       int shift = width % 8;
+       int shift = (8 - (width % 8)) & 7;
 
        width = (width + 7) & ~7;
 
@@ -85,7 +85,7 @@ static inline void rotate_cw(const char *in, char *out, u32 width, u32 height)
 static inline void rotate_ccw(const char *in, char *out, u32 width, u32 height)
 {
        int i, j, h = height, w = width;
-       int shift = width % 8;
+       int shift = (8 - (width % 8)) & 7;
 
        width = (width + 7) & ~7;
        height = (height + 7) & ~7;