diff options
author | Laurent Pinchart <laurent.pinchart@ideasonboard.com> | 2011-05-25 05:34:52 -0400 |
---|---|---|
committer | Laurent Pinchart <laurent.pinchart@ideasonboard.com> | 2011-08-19 04:31:09 -0400 |
commit | d8344e090c5adff3290925cbb040d20133b7f586 (patch) | |
tree | 55cc3ed84ceb1c73d274092135a03c55d2151835 /drivers/video | |
parent | 465cb7e700e8564fd2d76d5c7f3703eb07659617 (diff) |
g364fb: use display information in info not in var for panning
We must not use any information in the passed var besides xoffset,
yoffset and vmode as otherwise applications might abuse it.
Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Diffstat (limited to 'drivers/video')
-rw-r--r-- | drivers/video/g364fb.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/drivers/video/g364fb.c b/drivers/video/g364fb.c index d662317d85e3..223896cc5f7d 100644 --- a/drivers/video/g364fb.c +++ b/drivers/video/g364fb.c | |||
@@ -149,10 +149,11 @@ int g364fb_cursor(struct fb_info *info, struct fb_cursor *cursor) | |||
149 | static int g364fb_pan_display(struct fb_var_screeninfo *var, | 149 | static int g364fb_pan_display(struct fb_var_screeninfo *var, |
150 | struct fb_info *info) | 150 | struct fb_info *info) |
151 | { | 151 | { |
152 | if (var->xoffset || var->yoffset + var->yres > var->yres_virtual) | 152 | if (var->xoffset || |
153 | var->yoffset + info->var.yres > info->var.yres_virtual) | ||
153 | return -EINVAL; | 154 | return -EINVAL; |
154 | 155 | ||
155 | *(unsigned int *) TOP_REG = var->yoffset * var->xres; | 156 | *(unsigned int *) TOP_REG = var->yoffset * info->var.xres; |
156 | return 0; | 157 | return 0; |
157 | } | 158 | } |
158 | 159 | ||