diff options
| author | Kai Ye | 2022-11-12 02:12:50 +0000 |
|---|---|---|
| committer | Greg Kroah-Hartman | 2023-11-20 11:52:05 +0100 |
| commit | 56785a3a08e52252866411ae96735fcf8b4209cc (patch) | |
| tree | 32d6958b4890f2741e4e78238cae3a8d94a48f39 | |
| parent | bafb12b629b7c3ad59812dd1ac1b0618062e0e38 (diff) | |
crypto: hisilicon/qm - delete redundant null assignment operations
[ Upstream commit 7bbbc9d81be588ae4fb28b5b202e4421dbfef197 ]
There is no security data in the pointer. It is only a value transferred
as a structure. It makes no sense to zero a variable that is on the stack.
So not need to set the pointer to null.
Signed-off-by: Kai Ye <yekai13@huawei.com>
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
Stable-dep-of: 5831fc1fd4a5 ("crypto: hisilicon/qm - fix PF queue parameter issue")
Signed-off-by: Sasha Levin <sashal@kernel.org>
| -rw-r--r-- | drivers/crypto/hisilicon/qm.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/drivers/crypto/hisilicon/qm.c b/drivers/crypto/hisilicon/qm.c index 07e1e39a5e37..a878a232ef5b 100644 --- a/drivers/crypto/hisilicon/qm.c +++ b/drivers/crypto/hisilicon/qm.c @@ -1773,7 +1773,6 @@ static void dfx_regs_uninit(struct hisi_qm *qm, dregs[i].regs = NULL; } kfree(dregs); - dregs = NULL; } /** |