On Thu, Feb 13, 2014 at 8:51 PM, Paulo Zanoni <przanoni@xxxxxxxxx> wrote: > encoder = to_intel_encoder(connector->base.encoder); > + if (!encoder) > + continue; to_intel_connector can change a NULL pointer to something slightly different if the base structure isn't the very first thing. Hence you must do the check before the casting. I've fixed that up while applying, thanks for spotting and fixing this. -Daniel -- Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - http://blog.ffwll.ch _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx