Hi, On 1 June 2015 at 19:12, Maarten Lankhorst <maarten.lankhorst@xxxxxxxxxxxxxxx> wrote: > drm_framebuffer_reference(new_fb); > plane->fb = new_fb; > plane->crtc = plane->state->crtc; > + drm_framebuffer_unreference(plane->old_fb); NAK: this is missing a NULL-pointer check on old_fb. R-b stands if that's fixed though. Cheers, Daniel _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/dri-devel