]> git.baikalelectronics.ru Git - kernel.git/commit
padata: fix uninitialized return value in padata_replace()
authorDaniel Jordan <daniel.m.jordan@oracle.com>
Mon, 10 Feb 2020 18:11:00 +0000 (13:11 -0500)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Wed, 8 Apr 2020 07:08:40 +0000 (09:08 +0200)
commit485a875108cc7bf6ca75184630c125ea7c1b7ada
treef74f8c3082f7c3676b436c967cfbaf50dbd391cf
parenteb02f1e5659097a3cefa47e12d82aec773211313
padata: fix uninitialized return value in padata_replace()

[ Upstream commit e9d34c2a76dbbd6dfebf9008fc3354c828ffc977 ]

According to Geert's report[0],

  kernel/padata.c: warning: 'err' may be used uninitialized in this
    function [-Wuninitialized]:  => 539:2

Warning is seen only with older compilers on certain archs.  The
runtime effect is potentially returning garbage down the stack when
padata's cpumasks are modified before any pcrypt requests have run.

Simplest fix is to initialize err to the success value.

[0] http://lkml.kernel.org/r/20200210135506.11536-1-geert@linux-m68k.org

Reported-by: Geert Uytterhoeven <geert@linux-m68k.org>
Fixes: c7884cb49aa4 ("crypto: pcrypt - Avoid deadlock by using per-instance padata queues")
Signed-off-by: Daniel Jordan <daniel.m.jordan@oracle.com>
Cc: Herbert Xu <herbert@gondor.apana.org.au>
Cc: Steffen Klassert <steffen.klassert@secunet.com>
Cc: linux-crypto@vger.kernel.org
Cc: linux-kernel@vger.kernel.org
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
Signed-off-by: Sasha Levin <sashal@kernel.org>
kernel/padata.c