diff options
author | Dan Carpenter <dan.carpenter@oracle.com> | 2017-03-14 10:54:10 +0300 |
---|---|---|
committer | Gerd Hoffmann <kraxel@redhat.com> | 2017-03-14 11:54:20 +0100 |
commit | ee5cb7c465360f8eabc351196554e534675bdf6b (patch) | |
tree | 75d3ff400b1c35eebea621aecbe9039cea92adc2 /drivers/gpu/drm/qxl | |
parent | 43c27940227039223abbb5107a2eb3d799c8a2ec (diff) | |
download | op-kernel-dev-ee5cb7c465360f8eabc351196554e534675bdf6b.zip op-kernel-dev-ee5cb7c465360f8eabc351196554e534675bdf6b.tar.gz |
drm: qxl: add missing return check
My static checker complains that "release" is uninitialized if
qxl_alloc_release_reserved() fails, so let's add a check for that.
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Link: http://patchwork.freedesktop.org/patch/msgid/20170314075410.GB5984@mwanda
Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
Diffstat (limited to 'drivers/gpu/drm/qxl')
-rw-r--r-- | drivers/gpu/drm/qxl/qxl_display.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/drivers/gpu/drm/qxl/qxl_display.c b/drivers/gpu/drm/qxl/qxl_display.c index 9548bb5..058340a 100644 --- a/drivers/gpu/drm/qxl/qxl_display.c +++ b/drivers/gpu/drm/qxl/qxl_display.c @@ -572,6 +572,8 @@ static void qxl_cursor_atomic_update(struct drm_plane *plane, ret = qxl_alloc_release_reserved(qdev, sizeof(*cmd), QXL_RELEASE_CURSOR_CMD, &release, NULL); + if (ret) + return; cmd = (struct qxl_cursor_cmd *) qxl_release_map(qdev, release); |