diff options
| author | Dave Airlie <airlied@redhat.com> | 2009-04-20 01:32:50 +0200 |
|---|---|---|
| committer | Dave Airlie <airlied@redhat.com> | 2009-04-20 02:04:28 +0200 |
| commit | 07f1c7a7f6736d9ec2eba57d209c5f48888d841e (patch) | |
| tree | 0f75684fa43612f4f91d7aaeb91273f91097418f | |
| parent | drm: set/clear is_master when master changed (diff) | |
| download | linux-07f1c7a7f6736d9ec2eba57d209c5f48888d841e.tar.xz linux-07f1c7a7f6736d9ec2eba57d209c5f48888d841e.zip | |
drm: check for minor master before allowing drop master.
When fast user switching a lot eventually we get to the point,
where we were checking for the wrong thing in this function.
Signed-off-by: Dave Airlie <airlied@redhat.com>
| -rw-r--r-- | drivers/gpu/drm/drm_stub.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/drivers/gpu/drm/drm_stub.c b/drivers/gpu/drm/drm_stub.c index 1b3e0ff57b45..ef878615c49f 100644 --- a/drivers/gpu/drm/drm_stub.c +++ b/drivers/gpu/drm/drm_stub.c @@ -185,6 +185,9 @@ int drm_dropmaster_ioctl(struct drm_device *dev, void *data, if (!file_priv->is_master) return -EINVAL; + if (!file_priv->minor->master) + return -EINVAL; + mutex_lock(&dev->struct_mutex); drm_master_put(&file_priv->minor->master); file_priv->is_master = 0; |