diff options
| author | Heinrich Schuchardt <xypron.glpk@gmx.de> | 2020-04-19 12:02:28 +0200 |
|---|---|---|
| committer | Marek Vasut <marek.vasut+renesas@gmail.com> | 2020-04-28 13:52:52 +0200 |
| commit | cff0144e4cd0825dda0ae46aa739f0681839cc3a (patch) | |
| tree | f77d4031d941ae79497c3319240fbfe11a97298b /drivers/usb/host/ehci-hcd.c | |
| parent | d16d37bcd4087b8ea0f66cb76a73edad182d151a (diff) | |
usb: avoid NULL check before free
The free() function checks if the argument is NULL.
Do not duplicate this check.
Signed-off-by: Heinrich Schuchardt <xypron.glpk@gmx.de>
Diffstat (limited to 'drivers/usb/host/ehci-hcd.c')
| -rw-r--r-- | drivers/usb/host/ehci-hcd.c | 9 |
1 files changed, 3 insertions, 6 deletions
diff --git a/drivers/usb/host/ehci-hcd.c b/drivers/usb/host/ehci-hcd.c index 1edb344d0f..a2a85db1e7 100644 --- a/drivers/usb/host/ehci-hcd.c +++ b/drivers/usb/host/ehci-hcd.c @@ -1413,13 +1413,10 @@ static struct int_queue *_ehci_create_int_queue(struct usb_device *dev, debug("Exit create_int_queue\n"); return result; fail3: - if (result->tds) - free(result->tds); + free(result->tds); fail2: - if (result->first) - free(result->first); - if (result) - free(result); + free(result->first); + free(result); fail1: return NULL; } |