Commit a7b1ba23 authored by Suniel Mahesh's avatar Suniel Mahesh Committed by Greg Kroah-Hartman

staging: ccree: Fix unnecessary NULL check before kfree'ing it

kfree(NULL) is safe and their is no need for a NULL check. Pointed out
by checkpatch.
Signed-off-by: default avatarSuniel Mahesh <sunil.m@techveda.org>
Acked-by: default avatarGilad Ben-Yossef <gilad@benyossef.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 4a457c17
......@@ -297,20 +297,14 @@ static int ssi_hash_map_request(struct device *dev,
fail1:
kfree(state->digest_buff);
fail_digest_result_buff:
if (state->digest_result_buff) {
kfree(state->digest_result_buff);
state->digest_result_buff = NULL;
}
kfree(state->digest_result_buff);
state->digest_result_buff = NULL;
fail_buff1:
if (state->buff1) {
kfree(state->buff1);
state->buff1 = NULL;
}
kfree(state->buff1);
state->buff1 = NULL;
fail_buff0:
if (state->buff0) {
kfree(state->buff0);
state->buff0 = NULL;
}
kfree(state->buff0);
state->buff0 = NULL;
fail0:
return rc;
}
......@@ -2326,11 +2320,8 @@ int ssi_hash_alloc(struct ssi_drvdata *drvdata)
return 0;
fail:
if (drvdata->hash_handle) {
kfree(drvdata->hash_handle);
drvdata->hash_handle = NULL;
}
kfree(drvdata->hash_handle);
drvdata->hash_handle = NULL;
return rc;
}
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment