Thread (2 messages) 2 messages, 2 authors, 2014-05-23

Re: [PATCH] video: fbdev: grvga.c: Fix for possible null pointer dereference

flat view

From: Tomi Valkeinen <hidden>
Date: 2014-05-23 10:50:12
Also in: linux-devicetree, lkml

On 21/05/14 00:35, Rickard Strandqvist wrote:
quoted hunk ↗ jump to hunk
There is otherwise a risk of a possible null pointer dereference.

Was largely found by using a static code analysis program called cppcheck.

Signed-off-by: Rickard Strandqvist <redacted>
---
 drivers/video/fbdev/grvga.c |    3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/drivers/video/fbdev/grvga.c b/drivers/video/fbdev/grvga.c
index c078701..2db5bb1 100644
--- a/drivers/video/fbdev/grvga.c
+++ b/drivers/video/fbdev/grvga.c
@@ -514,9 +514,10 @@ free_fb:
 static int grvga_remove(struct platform_device *device)
 {
 	struct fb_info *info = dev_get_drvdata(&device->dev);
-	struct grvga_par *par = info->par;
+	struct grvga_par *par;
 
 	if (info) {
+		par = info->par;
 		unregister_framebuffer(info);
 		fb_dealloc_cmap(&info->cmap);
 
Thanks, queued this and the next one (video: fbdev: s3fb.c:  Fix for
possible null pointer dereference) for 3.16.

 Tomi


Attachments

Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help