Re: [PATCH] defxx: Fix a sentinel at the end of a 'eisa_device_id' structure
From: Maciej W. Rozycki
Date: Sun Feb 02 2020 - 14:12:22 EST
On Sun, 2 Feb 2020, Christophe JAILLET wrote:
> 'struct eisa_device_id' must be ended by an empty string, not a NULL
> pointer. Otherwise, a NULL pointer dereference may occur in
> 'eisa_bus_match()'.
Umm, that's weird code there in `eisa_bus_match' (I do hope at least GCC
optimises the `strlen' away nowadays and checks for the character pointed
being null instead), but as usually with old stuff let's keep changes to
the minimum. So:
Acked-by: Maciej W. Rozycki <macro@xxxxxxxxxxxxxx>
Thanks for the fix!
Maciej