[PATCH i-g-t 1/2] lib/igt_chamelium: check drmModeGetConnector() return value in wait_for_connected_state()
Kamil Konieczny
kamil.konieczny at linux.intel.com
Wed May 29 17:16:24 UTC 2024
Hi Luca,
On 2024-05-28 at 10:20:45 +0300, Luca Coelho wrote:
> On Fri, 2024-05-24 at 18:58 +0200, Kamil Konieczny wrote:
> > Hi Luca,
>
> Hi Kamil,
>
>
> > On 2024-05-23 at 11:44:25 +0300, Luca Coelho wrote:
> >
> > I have two nits, one about too long subject:
> >
> > [PATCH i-g-t 1/2] lib/igt_chamelium: check drmModeGetConnector() return value in wait_for_connected_state()
>
> Yes, it's a bit too long, but I wanted to convey useful information in
> the subject...
>
>
> > imho better:
> >
> > [PATCH i-g-t 1/2] lib/igt_chamelium: avoid NULL dereference in wait_for_connected_state()
>
> Okay, this is descriptive enough, but loses the context of the problem
> with drmModeGetConnector(), which I'm addressing in this series.
>
>
> > or even shorter:
> >
> > [PATCH i-g-t 1/2] lib/igt_chamelium: fix wait_for_connected_state()
>
> IMHO this is way too generic.
>
> What about this?
>
> [PATCH i-g-t 1/2] lib/igt_chamelium: check drmModeGetConnector() return in wait_for_connected_state()
imho still too long, another idea:
lib/igt_chamelium: check drmModeGetConnector() before using
Regards,
Kamil
>
> It's still sligthly over 80 chars, but conveys the full message (I just
> remove "value" from it). And the max 80 chars rule is so outdated that
> it's not very strict even in the kernel nowadays...
>
> > Maybe that should be changed to something else? See my other
> > response.
> >
> > > The drmModeGetConnector() function can return NULL in some cases, so
> > > we need to check the return value before accessing it. This is not
> > > being checked in wait_for_connected_state(), so fix that.
> > >
> >
> > Also please add to Cc chromium devs, for example:
> >
> > Cc: Mark Yacoub <markyacoub at chromium.org>
> > Cc: Manasi Navare <navaremanasi at chromium.org>
> > > Signed-off-by: Luca Coelho <luciano.coelho at intel.com>
>
> Done. Thanks for pointing out!
>
> --
> Cheers,
> Luca.
More information about the igt-dev
mailing list