[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20230110204418.79f43f45@kernel.org>
Date: Tue, 10 Jan 2023 20:44:18 -0800
From: Jakub Kicinski <kuba@...nel.org>
To: Esina Ekaterina <eesina@...ralinux.ru>
Cc: Zhao Qiang <qiang.zhao@....com>,
"David S . Miller" <davem@...emloft.net>,
Eric Dumazet <edumazet@...gle.com>,
Paolo Abeni <pabeni@...hat.com>, netdev@...r.kernel.org,
linuxppc-dev@...ts.ozlabs.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH v2] net: wan: Add checks for NULL. If uhdlc_priv_tsa !=
1 then utdm is not initialized. And if ret != NULL then goto
undo_uhdlc_init, where utdm is dereferenced. Same if dev == NULL.
On Tue, 10 Jan 2023 14:47:45 +0300 Esina Ekaterina wrote:
> Subject: [PATCH v2] net: wan: Add checks for NULL. If uhdlc_priv_tsa != 1 then utdm is not initialized. And if ret != NULL then goto undo_uhdlc_init, where utdm is dereferenced. Same if dev == NULL.
>
> Found by Linux Verification Center (linuxtesting.org) with SVACE.
>
> Signed-off-by: Esina Ekaterina <eesina@...ralinux.ru>
>
> v2: Add check for NULL for unmap_si_regs
> ---
The format of your commit is wrong. You should make the commit message
look like this in git:
net: wan: Add checks for NULL
If uhdlc_priv_tsa != 1 then utdm is not initialized.
And if ret != NULL then goto undo_uhdlc_init, where
utdm is dereferenced. Same if dev == NULL.
Found by Linux Verification Center (linuxtesting.org) with SVACE.
Signed-off-by: Esina Ekaterina <eesina@...ralinux.ru>
---
v2: Add check for NULL for unmap_si_regs
But the first line (subject) is still not specific enough.
Refer to the bug that's being fixed, not how it's fixed.
Also no braces needed around single-line if blocks.
Powered by blists - more mailing lists