diff options
author | Krzysztof Helt <krzysztof.h1@wp.pl> | 2009-12-15 19:46:25 -0500 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2009-12-16 10:20:01 -0500 |
commit | 2f390380ca69e1617cdddb12d8da94f0a9f4319d (patch) | |
tree | 2f60b69e62e099c43a90197c62704f099ad2169e /drivers/video/ep93xx-fb.c | |
parent | 2cb96f86628d6e97fcbda5fe4d8d74876239834c (diff) |
fbdev: add palette register check to several drivers
Add check if palette register number is in correct range for few drivers
which miss it. The regno value comes indirectly from user space.
Two drivers has converted check from BUG_ON() macro to just return an
error (non-zero value).
Signed-off-by: Krzysztof Helt <krzysztof.h1@wp.pl>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'drivers/video/ep93xx-fb.c')
-rw-r--r-- | drivers/video/ep93xx-fb.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/drivers/video/ep93xx-fb.c b/drivers/video/ep93xx-fb.c index bd9d46f95291..27aab4a06198 100644 --- a/drivers/video/ep93xx-fb.c +++ b/drivers/video/ep93xx-fb.c | |||
@@ -358,6 +358,8 @@ static int ep93xxfb_setcolreg(unsigned int regno, unsigned int red, | |||
358 | 358 | ||
359 | switch (info->fix.visual) { | 359 | switch (info->fix.visual) { |
360 | case FB_VISUAL_PSEUDOCOLOR: | 360 | case FB_VISUAL_PSEUDOCOLOR: |
361 | if (regno > 255) | ||
362 | return 1; | ||
361 | rgb = ((red & 0xff00) << 8) | (green & 0xff00) | | 363 | rgb = ((red & 0xff00) << 8) | (green & 0xff00) | |
362 | ((blue & 0xff00) >> 8); | 364 | ((blue & 0xff00) >> 8); |
363 | 365 | ||