diff options
author | Christophe JAILLET <christophe.jaillet@wanadoo.fr> | 2021-12-26 17:34:16 +0100 |
---|---|---|
committer | Daniel Vetter <daniel.vetter@ffwll.ch> | 2023-04-11 20:29:33 +0200 |
commit | b89ce1177d42d5c124e83f3858818cd4e6a2c46f (patch) | |
tree | 039492a61fd500f37e48538deb43d9064a1a6954 /drivers/gpu/drm/armada | |
parent | 6fd33a3333c7916689b8f051a185defe4dd515b0 (diff) | |
download | lwn-b89ce1177d42d5c124e83f3858818cd4e6a2c46f.tar.gz lwn-b89ce1177d42d5c124e83f3858818cd4e6a2c46f.zip |
drm/armada: Fix a potential double free in an error handling path
'priv' is a managed resource, so there is no need to free it explicitly or
there will be a double free().
Fixes: 90ad200b4cbc ("drm/armada: Use devm_drm_dev_alloc")
Signed-off-by: Christophe JAILLET <christophe.jaillet@wanadoo.fr>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Link: https://patchwork.freedesktop.org/patch/msgid/c4f3c9207a9fce35cb6dd2cc60e755275961588a.1640536364.git.christophe.jaillet@wanadoo.fr
Diffstat (limited to 'drivers/gpu/drm/armada')
-rw-r--r-- | drivers/gpu/drm/armada/armada_drv.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/drivers/gpu/drm/armada/armada_drv.c b/drivers/gpu/drm/armada/armada_drv.c index 0643887800b4..142668cd6d7c 100644 --- a/drivers/gpu/drm/armada/armada_drv.c +++ b/drivers/gpu/drm/armada/armada_drv.c @@ -99,7 +99,6 @@ static int armada_drm_bind(struct device *dev) if (ret) { dev_err(dev, "[" DRM_NAME ":%s] can't kick out simple-fb: %d\n", __func__, ret); - kfree(priv); return ret; } |