aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorZhang Qilong <[email protected]>2020-11-17 10:16:29 +0800
committerGreg Kroah-Hartman <[email protected]>2020-11-20 16:32:46 +0100
commit87bed3d7d26c974948a3d6e7176f304b2d41272b (patch)
treebb14545a03676673a3986f94722a55ed03b0a273
parente7694cb6998379341fd9bf3bd62b48c4e6a79385 (diff)
usb: gadget: Fix memleak in gadgetfs_fill_super
usb_get_gadget_udc_name will alloc memory for CHIP in "Enomem" branch. we should free it before error returns to prevent memleak. Fixes: 175f712119c57 ("usb: gadget: provide interface for legacy gadgets to get UDC name") Reported-by: Hulk Robot <[email protected]> Acked-by: Alan Stern <[email protected]> Signed-off-by: Zhang Qilong <[email protected]> Link: https://lore.kernel.org/r/[email protected] Cc: stable <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
-rw-r--r--drivers/usb/gadget/legacy/inode.c3
1 files changed, 3 insertions, 0 deletions
diff --git a/drivers/usb/gadget/legacy/inode.c b/drivers/usb/gadget/legacy/inode.c
index 1b430b36d0a6..71e7d10dd76b 100644
--- a/drivers/usb/gadget/legacy/inode.c
+++ b/drivers/usb/gadget/legacy/inode.c
@@ -2039,6 +2039,9 @@ gadgetfs_fill_super (struct super_block *sb, struct fs_context *fc)
return 0;
Enomem:
+ kfree(CHIP);
+ CHIP = NULL;
+
return -ENOMEM;
}