On Wed, Jan 18, 2017 at 12:49:00PM +0000, Chris Wilson wrote: > On Wed, Jan 18, 2017 at 02:34:28PM +0200, Ander Conselvan de Oliveira wrote: > > The error paths in hsw_trans_edp_pipe_A_crc_wa() and > > intel_prepare_reset() would potentially call drm_atomic_state_put with a > > NULL state, which would lead to a NULL pointer dereference. > > > > Found by coverity. > > > > v2: Improve the error paths. (Chris) > > > > Fixes: 0853695c3ba4 ("drm: Add reference counting to drm_atomic_state") > > Cc: Chris Wilson <ch...@chris-wilson.co.uk> > > Cc: Daniel Vetter <daniel.vet...@ffwll.ch> > > Cc: <drm-intel-fi...@lists.freedesktop.org> # v4.10-rc1+ > > Signed-off-by: Ander Conselvan de Oliveira > > <ander.conselvan.de.olive...@intel.com> > Reviewed-by: Chris Wilson <ch...@chris-wilson.co.uk>
Thanks for the patch for yet another of my mistakes. -Chris -- Chris Wilson, Intel Open Source Technology Centre _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx