summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorDan Carpenter <dan.carpenter@oracle.com>2013-05-30 10:00:34 +0200
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>2013-06-03 21:26:18 +0200
commit2a1afdbf20ef028d6361c2257b3fef51a8df8b36 (patch)
treefc2ca5c51a1f5e45aec4cc33784ec9b8ffb5a2e2
parentstaging: rts5139: Use correct USB transfer interval to fix syslog spamming (diff)
downloadlinux-2a1afdbf20ef028d6361c2257b3fef51a8df8b36.tar.xz
linux-2a1afdbf20ef028d6361c2257b3fef51a8df8b36.zip
Staging: crystalhd: remove an unneeded NULL check
We already established earlier in the function that "temp" is non-NULL. We also don't need to set to NULL because it's a stack variable an we return immediately. Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-rw-r--r--drivers/staging/crystalhd/crystalhd_lnx.c5
1 files changed, 1 insertions, 4 deletions
diff --git a/drivers/staging/crystalhd/crystalhd_lnx.c b/drivers/staging/crystalhd/crystalhd_lnx.c
index 284b61f53376..c1f6163cdeb8 100644
--- a/drivers/staging/crystalhd/crystalhd_lnx.c
+++ b/drivers/staging/crystalhd/crystalhd_lnx.c
@@ -255,10 +255,7 @@ static int chd_dec_api_cmd(struct crystalhd_adp *adp, unsigned long ua,
rc = chd_dec_proc_user_data(adp, temp, ua, 1);
}
- if (temp) {
- chd_dec_free_iodata(adp, temp, 0);
- temp = NULL;
- }
+ chd_dec_free_iodata(adp, temp, 0);
return rc;
}