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 | |
| 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>
| -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 | ||
