On Fri, 16 Jan 2026, "Kumar, Kaushlendra" <[email protected]> wrote: > On Fri, 16 Jan 2026, Jani Nikula wrote: > >> Do you have the backtrace? > > Hi Jani, > > I don't have a backtrace > intel_crtc_for_pipe() can return NULL, and dereferencing it without > checking seemed risky. If you'd prefer to see an actual crash before > applying, I understand. Happy to drop this if the NULL case isn't reachable > in practice..
Yeah, I'd like to know if this can actually happen, and if it has ever happened. If this is static analyzer inspired, there's the problem that IME adding NULL checks on these is a positive feedback loop to the analyzer that some places check for NULL i.e. not checking is an error. BR, Jani. -- Jani Nikula, Intel
