> Userspace can set a damage clip with a negative coordinate, negative > width or height or larger than the plane. > This invalid values could cause issues in some HW or even worst enable > security flaws. > > v2: > - add debug messages to let userspace know why atomic commit failed > due invalid damage clips > > Cc: Simon Ser <cont...@emersion.fr> > Cc: Gwan-gyeong Mun <gwan-gyeong....@intel.com> > Cc: Sean Paul <seanp...@chromium.org> > Cc: Fabio Estevam <feste...@gmail.com> > Cc: Deepak Rawat <dra...@vmware.com> > Cc: dri-devel@lists.freedesktop.org > Signed-off-by: José Roberto de Souza <jose.so...@intel.com>
After looking at the kernel code, it seems like the kernel already checks for all of that in drm_atomic_plane_check. Are you aware of this? > + w = drm_rect_width(&plane_state->src) >> 16; > + h = drm_rect_height(&plane_state->src) >> 16; The docs say this should be in FB coordinates, not in SRC_* coordinates. So we shouldn't need to check any SRC_* prop here. _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel
