On 28 March 2018 at 17:22, Emil Velikov <emil.l.velikov@xxxxxxxxx> wrote: > From: Emil Velikov <emil.velikov@xxxxxxxxxxxxx> > > Currently the error pointer returned by msm_alloc_stolen_fb gets passed > to drm_framebuffer_remove. The latter handles only NULL pointers, thus > a nasty crash will occur. > > Drop the unnecessary fail label and the associated checks - both err and > fb will be set at this stage. > > Cc: Rob Clark <robdclark@xxxxxxxxx> > Cc: linux-arm-msm@xxxxxxxxxxxxxxx > Cc: dri-devel@xxxxxxxxxxxxxxxxxxxxx > Cc: freedreno@xxxxxxxxxxxxxxxxxxxxx > Signed-off-by: Emil Velikov <emil.velikov@xxxxxxxxxxxxx> > --- > v2: really remove the fail label :-\ > --- Humble ping? -Emil _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/dri-devel
