MIRRORS/linux-imx
1
0
Fork 0
mirror of https://github.com/nxp-imx/linux-imx.git synced 2025-09-03 10:33:11 +02:00
Code Issues Actions Packages Projects Releases Wiki Activity

MLK-20204: drivers: crypto: caam: sm: Remove deadcode

Browse Source 
There was some code to free the keystore but the current
state of the function make this code unreachable so
remove it.

Coverity explanation:
drivers/crypto/caam/sm_store.c:654: CID 17839 (#1 of 1):
  Type: Logically dead code (DEADCODE)
  Classification: Bug
  Severity: Major
  Action: Fix Required
  Owner: nxa21133
  Defect only exists locally.
drivers/crypto/caam/sm_store.c:625:
  cond_null: Condition "keystore_data == NULL", taking true branch. Now
  the value of "keystore_data" is "NULL".
drivers/crypto/caam/sm_store.c:653:
  null: At condition "keystore_data != NULL", the value of "keystore_data"
  must be "NULL".
drivers/crypto/caam/sm_store.c:653:
  dead_error_condition: The condition "keystore_data != NULL" cannot be
  true.
drivers/crypto/caam/sm_store.c:654:
  dead_error_line: Execution cannot reach this statement:
  "kfree(keystore_data);".

Signed-off-by: Franck LENORMAND <franck.lenormand@nxp.com>
This commit is contained in:
Franck LENORMAND 2018-11-23 11:37:28 +01:00 committed by Leonard Crestez
parent 09030c7e27
commit b7385ab947
1 changed files with 3 additions and 14 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

17
drivers/crypto/caam/sm_store.c
View File

@ -605,7 +605,6 @@ u32 slot_get_slot_size(struct device *dev, u32 unit, u32 slot)
int kso_init_data(struct device *dev, u32 unit)
{
struct caam_drv_private_sm *smpriv = dev_get_drvdata(dev);
int retval = -EINVAL;
struct keystore_data *keystore_data = NULL;
u32 slot_count;
u32 keystore_data_size;
@ -625,10 +624,8 @@ int kso_init_data(struct device *dev, u32 unit)
keystore_data = kzalloc(keystore_data_size, GFP_KERNEL);
if (keystore_data == NULL) {
retval = -ENOSPC;
goto out;
}
if (!keystore_data)
return -ENOMEM;
#ifdef SM_DEBUG
dev_info(dev, "kso_init_data: keystore data size = %d\n",
@ -649,15 +646,7 @@ int kso_init_data(struct device *dev, u32 unit)
smpriv->pagedesc[unit].ksdata->phys_address =
smpriv->pagedesc[unit].pg_phys;
retval = 0;
out:
if (retval != 0)
if (keystore_data != NULL)
kfree(keystore_data);
return retval;
return 0;
}
void kso_cleanup_data(struct device *dev, u32 unit)