mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
* [PATCH] Radeon driver: fix memory leak
@ 2013-08-21 18:31 Filipe David Borba Manana
  2013-08-21 18:53 ` [PATCH v2] " Filipe David Borba Manana
  0 siblings, 1 reply; 2+ messages in thread
From: Filipe David Borba Manana @ 2013-08-21 18:31 UTC (permalink / raw)
  To: jglisse; +Cc: linux-kernel, Filipe David Borba Manana

When freeing a struct radeon_device, its member iio was
not being freed, resulting in a memory leak.

Signed-off-by: Filipe David Borba Manana <fdmanana@gmail.com>
---
 drivers/gpu/drm/radeon/radeon_device.c |    1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/gpu/drm/radeon/radeon_device.c b/drivers/gpu/drm/radeon/radeon_device.c
index 63398ae..e57efb4 100644
--- a/drivers/gpu/drm/radeon/radeon_device.c
+++ b/drivers/gpu/drm/radeon/radeon_device.c
@@ -912,6 +912,7 @@ void radeon_atombios_fini(struct radeon_device *rdev)
 	if (rdev->mode_info.atom_context) {
 		kfree(rdev->mode_info.atom_context->scratch);
 	}
+	kfree(rdev->mode_info.atom_context->iio);
 	kfree(rdev->mode_info.atom_context);
 	rdev->mode_info.atom_context = NULL;
 	kfree(rdev->mode_info.atom_card_info);
-- 
1.7.9.5


^ permalink raw reply	[flat|nested] 2+ messages in thread

* [PATCH v2] Radeon driver: fix memory leak
  2013-08-21 18:31 [PATCH] Radeon driver: fix memory leak Filipe David Borba Manana
@ 2013-08-21 18:53 ` Filipe David Borba Manana
  0 siblings, 0 replies; 2+ messages in thread
From: Filipe David Borba Manana @ 2013-08-21 18:53 UTC (permalink / raw)
  To: jglisse; +Cc: linux-kernel, Filipe David Borba Manana

When freeing a struct radeon_device, its member iio was
not being freed, resulting in a memory leak.

Signed-off-by: Filipe David Borba Manana <fdmanana@gmail.com>
---

V2: put kfree() call inside if statement.

 drivers/gpu/drm/radeon/radeon_device.c |    1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/gpu/drm/radeon/radeon_device.c b/drivers/gpu/drm/radeon/radeon_device.c
index 63398ae..5d94d5e 100644
--- a/drivers/gpu/drm/radeon/radeon_device.c
+++ b/drivers/gpu/drm/radeon/radeon_device.c
@@ -911,6 +911,7 @@ void radeon_atombios_fini(struct radeon_device *rdev)
 {
 	if (rdev->mode_info.atom_context) {
 		kfree(rdev->mode_info.atom_context->scratch);
+		kfree(rdev->mode_info.atom_context->iio);
 	}
 	kfree(rdev->mode_info.atom_context);
 	rdev->mode_info.atom_context = NULL;
-- 
1.7.9.5


^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2013-08-21 18:54 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-08-21 18:31 [PATCH] Radeon driver: fix memory leak Filipe David Borba Manana
2013-08-21 18:53 ` [PATCH v2] " Filipe David Borba Manana

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®