Re: [PATCH next] efivarfs: Fix a NULL vs IS_ERR() bug

From: Dan Carpenter
Date: Mon Jan 13 2025 - 09:26:16 EST


On Mon, Jan 13, 2025 at 03:18:05PM +0100, Ard Biesheuvel wrote:
> On Mon, 13 Jan 2025 at 07:16, Dan Carpenter <dan.carpenter@xxxxxxxxxx> wrote:
> >
> > The kernel_file_open() function doesn't return NULL, it retursn error
> > pointers. Fix the error checking to match.
> >
> > Fixes: eb11e99adef8 ("efivarfs: add variable resync after hibernation")
> > Signed-off-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx>
> > ---
> > fs/efivarfs/super.c | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
>
> Thanks - I'll fold this in rather than apply it on top though.

Sure. No worries.

regards,
dan carpenter