| Date | Sun, 22 Sep 2019 22:26:11 +0200 | From | Greg KH <> | Subject | Re: [PATCH AUTOSEL 5.2 154/185] x86/platform/uv: Fix kmalloc() NULL check routine |
| |
On Sun, Sep 22, 2019 at 02:48:52PM -0400, Sasha Levin wrote: > From: Austin Kim <austindh.kim@gmail.com> > > [ Upstream commit 864b23f0169d5bff677e8443a7a90dfd6b090afc ] > > The result of kmalloc() should have been checked ahead of below statement: > > pqp = (struct bau_pq_entry *)vp; > > Move BUG_ON(!vp) before above statement. > > Signed-off-by: Austin Kim <austindh.kim@gmail.com> > Cc: Dimitri Sivanich <dimitri.sivanich@hpe.com> > Cc: Hedi Berriche <hedi.berriche@hpe.com> > Cc: Linus Torvalds <torvalds@linux-foundation.org> > Cc: Mike Travis <mike.travis@hpe.com> > Cc: Peter Zijlstra <peterz@infradead.org> > Cc: Russ Anderson <russ.anderson@hpe.com> > Cc: Steve Wahl <steve.wahl@hpe.com> > Cc: Thomas Gleixner <tglx@linutronix.de> > Cc: allison@lohutok.net > Cc: andy@infradead.org > Cc: armijn@tjaldur.nl > Cc: bp@alien8.de > Cc: dvhart@infradead.org > Cc: gregkh@linuxfoundation.org > Cc: hpa@zytor.com > Cc: kjlu@umn.edu > Cc: platform-driver-x86@vger.kernel.org > Link: https://lkml.kernel.org/r/20190905232951.GA28779@LGEARND20B15 > Signed-off-by: Ingo Molnar <mingo@kernel.org> > Signed-off-by: Sasha Levin <sashal@kernel.org> > --- > arch/x86/platform/uv/tlb_uv.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/arch/x86/platform/uv/tlb_uv.c b/arch/x86/platform/uv/tlb_uv.c > index 0c7dfec4acac8..5a66d73620ce4 100644 > --- a/arch/x86/platform/uv/tlb_uv.c > +++ b/arch/x86/platform/uv/tlb_uv.c > @@ -1815,9 +1815,9 @@ static void pq_init(int node, int pnode) > > plsize = (DEST_Q_SIZE + 1) * sizeof(struct bau_pq_entry); > vp = kmalloc_node(plsize, GFP_KERNEL, node); > - pqp = (struct bau_pq_entry *)vp; > - BUG_ON(!pqp); > + BUG_ON(!vp); > > + pqp = (struct bau_pq_entry *)vp; > cp = (char *)pqp + 31; > pqp = (struct bau_pq_entry *)(((unsigned long)cp >> 5) << 5); >
Please drop from everywhere.
thanks,
greg k-h
|