diff options
| author | Daniel Vetter | 2014-04-05 11:12:04 +0200 |
|---|---|---|
| committer | Daniel Vetter | 2014-04-22 15:39:42 +0200 |
| commit | 10e6856983c21b76d03282b6da86709bdc23eb77 (patch) | |
| tree | 4f98c7f11c36938a5976036c4b58128f2e289b13 /drivers/gpu/drm/drm_stub.c | |
| parent | drm/i2c/tda998x: Fix signed overflow issue (diff) | |
| download | kernel-qcow2-linux-10e6856983c21b76d03282b6da86709bdc23eb77.tar.gz kernel-qcow2-linux-10e6856983c21b76d03282b6da86709bdc23eb77.tar.xz kernel-qcow2-linux-10e6856983c21b76d03282b6da86709bdc23eb77.zip | |
drm: Fix error handling in drm_master_create
We need to check whether drm_ht_create succeed and clean up
if not.
Spotted by coverity.
Reviewed-by: David Herrmann <dh.herrmann@gmail.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Diffstat (limited to 'drivers/gpu/drm/drm_stub.c')
| -rw-r--r-- | drivers/gpu/drm/drm_stub.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/drivers/gpu/drm/drm_stub.c b/drivers/gpu/drm/drm_stub.c index 4c24c3ac1efa..bfa6cb949545 100644 --- a/drivers/gpu/drm/drm_stub.c +++ b/drivers/gpu/drm/drm_stub.c @@ -128,7 +128,10 @@ struct drm_master *drm_master_create(struct drm_minor *minor) kref_init(&master->refcount); spin_lock_init(&master->lock.spinlock); init_waitqueue_head(&master->lock.lock_queue); - drm_ht_create(&master->magiclist, DRM_MAGIC_HASH_ORDER); + if (drm_ht_create(&master->magiclist, DRM_MAGIC_HASH_ORDER)) { + kfree(master); + return NULL; + } INIT_LIST_HEAD(&master->magicfree); master->minor = minor; |