| From 80b15db5e1e9c3300de299b2d43d1aafb593e6ac Mon Sep 17 00:00:00 2001 |
| From: Navid Emamdoost <navid.emamdoost@gmail.com> |
| Date: Fri, 4 Oct 2019 15:03:15 -0500 |
| Subject: staging: vt6655: Fix memory leak in vt6655_probe |
| |
| From: Navid Emamdoost <navid.emamdoost@gmail.com> |
| |
| commit 80b15db5e1e9c3300de299b2d43d1aafb593e6ac upstream. |
| |
| In vt6655_probe, if vnt_init() fails the cleanup code needs to be called |
| like other error handling cases. The call to device_free_info() is |
| added. |
| |
| Fixes: 67013f2c0e58 ("staging: vt6655: mac80211 conversion add main mac80211 functions") |
| Signed-off-by: Navid Emamdoost <navid.emamdoost@gmail.com> |
| Cc: stable <stable@vger.kernel.org> |
| Link: https://lore.kernel.org/r/20191004200319.22394-1-navid.emamdoost@gmail.com |
| Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> |
| |
| --- |
| drivers/staging/vt6655/device_main.c | 4 +++- |
| 1 file changed, 3 insertions(+), 1 deletion(-) |
| |
| --- a/drivers/staging/vt6655/device_main.c |
| +++ b/drivers/staging/vt6655/device_main.c |
| @@ -1668,8 +1668,10 @@ vt6655_probe(struct pci_dev *pcid, const |
| |
| priv->hw->max_signal = 100; |
| |
| - if (vnt_init(priv)) |
| + if (vnt_init(priv)) { |
| + device_free_info(priv); |
| return -ENODEV; |
| + } |
| |
| device_print_info(priv); |
| pci_set_drvdata(pcid, priv); |
