Op 10-04-17 om 13:54 schreef Daniel Vetter: > Yet again I've proven that I can't negate conditions :( > > Fixes: eb8eb02ed850 ("drm: Drop modeset_lock_all from the getproperty ioctl") > Cc: Maarten Lankhorst <maarten.lankhorst@xxxxxxxxxxxxxxx> > Cc: Daniel Vetter <daniel.vetter@xxxxxxxxx> > Cc: Jani Nikula <jani.nikula@xxxxxxxxxxxxxxx> > Cc: Sean Paul <seanpaul@xxxxxxxxxxxx> > Reported-by: Tvrtko Ursulin <tvrtko.ursulin@xxxxxxxxx> > Cc: Tvrtko Ursulin <tvrtko.ursulin@xxxxxxxxx> > Signed-off-by: Daniel Vetter <daniel.vetter@xxxxxxxxx> > --- > drivers/gpu/drm/drm_property.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/gpu/drm/drm_property.c b/drivers/gpu/drm/drm_property.c > index 3feef0659940..3e88fa24eab3 100644 > --- a/drivers/gpu/drm/drm_property.c > +++ b/drivers/gpu/drm/drm_property.c > @@ -476,7 +476,7 @@ int drm_mode_getproperty_ioctl(struct drm_device *dev, > drm_property_type_is(property, DRM_MODE_PROP_BITMASK)) { > list_for_each_entry(prop_enum, &property->enum_list, head) { > enum_count++; > - if (out_resp->count_enum_blobs <= enum_count) > + if (out_resp->count_enum_blobs < enum_count) > continue; > > if (copy_to_user(&enum_ptr[copied].value, Neither can I, glanced over it while looking why the bisect pointed at this commit. Reviewed-by: Maarten Lankhorst <maarten.lankhorst@xxxxxxxxxxxxxxx> _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/dri-devel