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