[PATCH] Fix leak in aty fb code. (fwd)

STALE8040d

2 messages, 2 authors, 2004-09-02 · open the first message on its own page

[PATCH] Fix leak in aty fb code. (fwd)

From: Geert Uytterhoeven <geert@linux-m68k.org>
Date: 2004-09-02 09:44:57 

---------- Forwarded message ----------
Date: Wed, 1 Sep 2004 16:51:21 +0100
From: Dave Jones <redacted>
To: linux-kernel@vger.kernel.org
Subject: [PATCH] Fix leak in aty fb code.

Spotted with the source checker from Coverity.com.

Signed-off-by: Dave Jones <redacted>


diff -urpN --exclude-from=/home/davej/.exclude bk-linus/drivers/video/aty/atyfb_base.c linux-2.6/drivers/video/aty/atyfb_base.c
--- bk-linus/drivers/video/aty/atyfb_base.c	2004-08-01 00:00:35.000000000 +0100
+++ linux-2.6/drivers/video/aty/atyfb_base.c	2004-08-23 14:08:20.000000000 +0100
@@ -2374,6 +2374,7 @@ int __init atyfb_init(void)
 		}
 	}
 #endif				/* CONFIG_ATARI */
+	kfree(info);
 	return 0;
 }

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


-------------------------------------------------------
This SF.Net email is sponsored by BEA Weblogic Workshop
FREE Java Enterprise J2EE developer tools!
Get your free copy of BEA WebLogic Workshop 8.1 today.
http://ads.osdn.com/?ad_id=5047&alloc_id=10808&op=click

Re: [PATCH] Fix leak in aty fb code. (fwd)

From: Antonino A. Daplas <hidden>
Date: 2004-09-02 20:56:55 

On Thursday 02 September 2004 17:44, Geert Uytterhoeven wrote:
quoted hunk
---------- Forwarded message ----------
Date: Wed, 1 Sep 2004 16:51:21 +0100
From: Dave Jones <redacted>
To: linux-kernel@vger.kernel.org
Subject: [PATCH] Fix leak in aty fb code.

Spotted with the source checker from Coverity.com.

Signed-off-by: Dave Jones <redacted>


diff -urpN --exclude-from=/home/davej/.exclude
bk-linus/drivers/video/aty/atyfb_base.c
linux-2.6/drivers/video/aty/atyfb_base.c ---
bk-linus/drivers/video/aty/atyfb_base.c	2004-08-01 00:00:35.000000000 +0100
+++ linux-2.6/drivers/video/aty/atyfb_base.c	2004-08-23 14:08:20.000000000
+0100 @@ -2374,6 +2374,7 @@ int __init atyfb_init(void)
 		}
 	}
 #endif				/* CONFIG_ATARI */
+	kfree(info);
 	return 0;
 }
This is also incorrect, and will actually cause a crash. This is part of
init code, and info should not be freed.  It will be freed in
cleanup_module().

Tony




-------------------------------------------------------
This SF.Net email is sponsored by BEA Weblogic Workshop
FREE Java Enterprise J2EE developer tools!
Get your free copy of BEA WebLogic Workshop 8.1 today.
http://ads.osdn.com/?ad_id=5047&alloc_id=10808&op=click
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help