vt8623fb: arkfb: null pointer dereference fix

PATCH In review Antonino A. Daplas 1 participant 1 message May 27, 2007 Framebuffer layer

lore

Share & tools

lore

Antonino A. Daplas

author

2007-05-27 09:51 UTC #0 lore raw

PATCH 2/9 vt8623fb: arkfb: null pointer dereference fix

From: Ondrej Zajicek <[email protected]>

This patch prevents null pointer dereference in arkfb and vt8623fb.

Signed-off-by Ondrej Zajicek <[email protected]>

Signed-off-by Antonino Daplas <[email protected]>

---

 drivers/video/arkfb.c    |    3 ++-
 drivers/video/vt8623fb.c |    3 ++-
 2 files changed, 4 insertions(+), 2 deletions(-)

drivers/video/arkfb.c +2 -1

index ba6fede..8a1b07c 100644
@@ -1055,9 +1055,10 @@ err_enable_device:
 static void __devexit ark_pci_remove(struct pci_dev *dev)
 {
 	struct fb_info *info = pci_get_drvdata(dev);
-	struct arkfb_info *par = info->par;
 
 	if (info) {
+		struct arkfb_info *par = info->par;
+
 #ifdef CONFIG_MTRR
 		if (par->mtrr_reg >= 0) {
 			mtrr_del(par->mtrr_reg, 0, 0);

drivers/video/vt8623fb.c +2 -1

index 5e9755e..30c0b94 100644
@@ -778,9 +778,10 @@ err_enable_device:
 static void __devexit vt8623_pci_remove(struct pci_dev *dev)
 {
 	struct fb_info *info = pci_get_drvdata(dev);
-	struct vt8623fb_info *par = info->par;
 
 	if (info) {
+		struct vt8623fb_info *par = info->par;
+
 #ifdef CONFIG_MTRR
 		if (par->mtrr_reg >= 0) {
 			mtrr_del(par->mtrr_reg, 0, 0);

-------------------------------------------------------------------------
This SF.net email is sponsored by DB2 Express
Download DB2 Express C - the FREE version of DB2 express and take control of your XML. No limits. Just data. Click to get it now.
http://sourceforge.net/powerbar/db2/