diff options
| author | Antonino A. Daplas <adaplas@gmail.com> | 2005-09-09 16:10:02 -0400 |
|---|---|---|
| committer | Linus Torvalds <torvalds@g5.osdl.org> | 2005-09-09 17:03:40 -0400 |
| commit | 61ab7903b8cd772d3bfb28bc26d02c599cfb0e5b (patch) | |
| tree | 4140b59ec3882a954b620ebbee56d6c7bcdf781d /drivers | |
| parent | 7c1cd6fd5efeb95603e37f35b5da293b452d8b64 (diff) | |
[PATCH] fbdev: Initialize var structure in calc_mode_timings
The var structure in calc_mode_timings is not properly initialized (zero set)
which leads to undefined behavior when it is passed to fb_get_mode().
Signed-off-by: Antonino Daplas <adaplas@pol.net>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
Diffstat (limited to 'drivers')
| -rw-r--r-- | drivers/video/fbmon.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/drivers/video/fbmon.c b/drivers/video/fbmon.c index 791bec3d672a..713226cdf3c6 100644 --- a/drivers/video/fbmon.c +++ b/drivers/video/fbmon.c | |||
| @@ -314,11 +314,13 @@ static int edid_is_monitor_block(unsigned char *block) | |||
| 314 | return 0; | 314 | return 0; |
| 315 | } | 315 | } |
| 316 | 316 | ||
| 317 | static void calc_mode_timings(int xres, int yres, int refresh, struct fb_videomode *mode) | 317 | static void calc_mode_timings(int xres, int yres, int refresh, |
| 318 | struct fb_videomode *mode) | ||
| 318 | { | 319 | { |
| 319 | struct fb_var_screeninfo var; | 320 | struct fb_var_screeninfo var; |
| 320 | struct fb_info info; | 321 | struct fb_info info; |
| 321 | 322 | ||
| 323 | memset(&var, 0, sizeof(struct fb_var_screeninfo)); | ||
| 322 | var.xres = xres; | 324 | var.xres = xres; |
| 323 | var.yres = yres; | 325 | var.yres = yres; |
| 324 | fb_get_mode(FB_VSYNCTIMINGS | FB_IGNOREMON, | 326 | fb_get_mode(FB_VSYNCTIMINGS | FB_IGNOREMON, |
