diff options
author | Dan Carpenter <dan.carpenter@oracle.com> | 2013-11-12 05:25:26 -0500 |
---|---|---|
committer | Thierry Reding <treding@nvidia.com> | 2013-12-20 09:56:06 -0500 |
commit | 6b7c79d19157d52901398964ad115dd472cedfde (patch) | |
tree | c0220a19cc5436d33a61e3f47ac0942e3bb4f5cf /drivers/gpu/drm/tegra | |
parent | 60c2f709d9b41184e0279d7538a5217885da8dde (diff) |
drm/tegra: Fix small leak on error in tegra_fb_alloc()
If we don't have enough memory for ->planes then we leak "fb".
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Thierry Reding <treding@nvidia.com>
Diffstat (limited to 'drivers/gpu/drm/tegra')
-rw-r--r-- | drivers/gpu/drm/tegra/fb.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/drivers/gpu/drm/tegra/fb.c b/drivers/gpu/drm/tegra/fb.c index bd4bf210f469..f7fca09d4921 100644 --- a/drivers/gpu/drm/tegra/fb.c +++ b/drivers/gpu/drm/tegra/fb.c | |||
@@ -100,8 +100,10 @@ static struct tegra_fb *tegra_fb_alloc(struct drm_device *drm, | |||
100 | return ERR_PTR(-ENOMEM); | 100 | return ERR_PTR(-ENOMEM); |
101 | 101 | ||
102 | fb->planes = kzalloc(num_planes * sizeof(*planes), GFP_KERNEL); | 102 | fb->planes = kzalloc(num_planes * sizeof(*planes), GFP_KERNEL); |
103 | if (!fb->planes) | 103 | if (!fb->planes) { |
104 | kfree(fb); | ||
104 | return ERR_PTR(-ENOMEM); | 105 | return ERR_PTR(-ENOMEM); |
106 | } | ||
105 | 107 | ||
106 | fb->num_planes = num_planes; | 108 | fb->num_planes = num_planes; |
107 | 109 | ||