summaryrefslogtreecommitdiffstats
path: root/drivers/video
diff options
context:
space:
mode:
authorAntonino A. Daplas <adaplas@gmail.com>2005-09-14 14:19:15 -0700
committerLinus Torvalds <torvalds@g5.osdl.org>2005-09-14 14:34:17 -0700
commit0ed8e048c9e11e69ec951f94066105e80cdc59fd (patch)
treeaaebd53676e1629c57cbbb90f24d1a4d6ad598d6 /drivers/video
parent8fd9808aec0198b25ff9f8bb19508d9492f86920 (diff)
downloadlinux-0ed8e048c9e11e69ec951f94066105e80cdc59fd.tar.gz
linux-0ed8e048c9e11e69ec951f94066105e80cdc59fd.tar.bz2
linux-0ed8e048c9e11e69ec951f94066105e80cdc59fd.zip
[PATCH] nv_i2c oops fix
The call to fb_firmware_edid may return NULL but this is not checked before trying to memcpy using this pointer. 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/video')
-rw-r--r--drivers/video/nvidia/nv_i2c.c11
1 files changed, 7 insertions, 4 deletions
diff --git a/drivers/video/nvidia/nv_i2c.c b/drivers/video/nvidia/nv_i2c.c
index ace484fa61ce..12f2884d3f0b 100644
--- a/drivers/video/nvidia/nv_i2c.c
+++ b/drivers/video/nvidia/nv_i2c.c
@@ -209,10 +209,13 @@ int nvidia_probe_i2c_connector(struct fb_info *info, int conn, u8 **out_edid)
if (!edid && conn == 1) {
/* try to get from firmware */
- edid = kmalloc(EDID_LENGTH, GFP_KERNEL);
- if (edid)
- memcpy(edid, fb_firmware_edid(info->device),
- EDID_LENGTH);
+ const u8 *e = fb_firmware_edid(info->device);
+
+ if (e != NULL) {
+ edid = kmalloc(EDID_LENGTH, GFP_KERNEL);
+ if (edid)
+ memcpy(edid, e, EDID_LENGTH);
+ }
}
if (out_edid)