mirror of
https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git
synced 2024-08-21 16:29:59 +00:00
crypto: qat - Deletion of unnecessary checks before two function calls
The functions kfree() and release_firmware() test whether their argument is NULL and then return immediately. Thus the test around the calls is not needed. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring <elfring@users.sourceforge.net> Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
This commit is contained in:
parent
44a17ef872
commit
12a4bd3122
2 changed files with 2 additions and 5 deletions
|
@ -88,10 +88,7 @@ void adf_ae_fw_release(struct adf_accel_dev *accel_dev)
|
|||
|
||||
qat_uclo_del_uof_obj(loader_data->fw_loader);
|
||||
qat_hal_deinit(loader_data->fw_loader);
|
||||
|
||||
if (loader_data->uof_fw)
|
||||
release_firmware(loader_data->uof_fw);
|
||||
|
||||
release_firmware(loader_data->uof_fw);
|
||||
loader_data->uof_fw = NULL;
|
||||
loader_data->fw_loader = NULL;
|
||||
}
|
||||
|
|
|
@ -449,7 +449,7 @@ static int adf_init_bank(struct adf_accel_dev *accel_dev,
|
|||
err:
|
||||
for (i = 0; i < ADF_ETR_MAX_RINGS_PER_BANK; i++) {
|
||||
ring = &bank->rings[i];
|
||||
if (hw_data->tx_rings_mask & (1 << i) && ring->inflights)
|
||||
if (hw_data->tx_rings_mask & (1 << i))
|
||||
kfree(ring->inflights);
|
||||
}
|
||||
return -ENOMEM;
|
||||
|
|
Loading…
Reference in a new issue