diff options
| author | Markus Elfring <elfring@users.sourceforge.net> | 2014-12-01 15:49:39 -0300 |
|---|---|---|
| committer | Mauro Carvalho Chehab <mchehab@osg.samsung.com> | 2014-12-04 15:47:06 -0200 |
| commit | e8379ecff31c0a172cb2ccc7cf28c24b45f144cf (patch) | |
| tree | 8a0463c29e874e94b8f1882a4b3996963ed7120c | |
| parent | 9a4ea5a98652f602d4ec16957f64fd666e862b09 (diff) | |
| download | op-kernel-dev-e8379ecff31c0a172cb2ccc7cf28c24b45f144cf.zip op-kernel-dev-e8379ecff31c0a172cb2ccc7cf28c24b45f144cf.tar.gz | |
[media] lirc_zilog: Deletion of unnecessary checks before vfree()
The vfree() function performs also input parameter validation. Thus the test
around the call is not needed.
This issue was detected by using the Coccinelle software.
Signed-off-by: Markus Elfring <elfring@users.sourceforge.net>
Signed-off-by: Mauro Carvalho Chehab <mchehab@osg.samsung.com>
| -rw-r--r-- | drivers/staging/media/lirc/lirc_zilog.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/drivers/staging/media/lirc/lirc_zilog.c b/drivers/staging/media/lirc/lirc_zilog.c index 1ccf626..1e15d2c 100644 --- a/drivers/staging/media/lirc/lirc_zilog.c +++ b/drivers/staging/media/lirc/lirc_zilog.c @@ -730,11 +730,9 @@ static int send_boot_data(struct IR_tx *tx) static void fw_unload_locked(void) { if (tx_data) { - if (tx_data->code_sets) - vfree(tx_data->code_sets); + vfree(tx_data->code_sets); - if (tx_data->datap) - vfree(tx_data->datap); + vfree(tx_data->datap); vfree(tx_data); tx_data = NULL; |
