Hello,

On Tue, Jan 6, 2009 at 12:34 AM, Eric Anholt <e...@anholt.net> wrote:
> Signed-off-by: Eric Anholt <e...@anholt.net>
> ---
>  drivers/gpu/drm/i915/i915_gem.c |    7 +++++++
>  1 files changed, 7 insertions(+), 0 deletions(-)
>
> diff --git a/drivers/gpu/drm/i915/i915_gem.c b/drivers/gpu/drm/i915/i915_gem.c
> index 14afc23..e87db6f 100644
> --- a/drivers/gpu/drm/i915/i915_gem.c
> +++ b/drivers/gpu/drm/i915/i915_gem.c
> @@ -2761,6 +2761,13 @@ i915_gem_unpin_ioctl(struct drm_device *dev, void 
> *data,
>                mutex_unlock(&dev->struct_mutex);
>                return -EBADF;
>        }
> +       obj_priv = obj->driver_private;
> +
> +       if (obj_priv->pin_count == 0) {
> +               drm_gem_object_unreference(obj);
> +               mutex_unlock(&dev->struct_mutex);
> +               return -EINVAL;
> +       }
>
>        obj_priv = obj->driver_private;

Is it necessary to do this again?


>        if (obj_priv->pin_filp != file_priv) {
> --
> 1.5.6.5
>
>

Cheers,
  Renato

------------------------------------------------------------------------------
--
_______________________________________________
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel

Reply via email to