summaryrefslogtreecommitdiffstats
path: root/drivers/usb/gadget
diff options
context:
space:
mode:
authorWei Yongjun <yongjun_wei@trendmicro.com.cn>2013-06-18 11:40:55 +0800
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>2013-06-18 10:50:08 -0700
commitdf48fc74bffe09675b43aa2bd4b00c8cbb37e653 (patch)
treee02c291b011e558ecac71bd86133fd9823d2a1ea /drivers/usb/gadget
parentc92834c19fc63f8a1d344eee0dd3d62c7b449e26 (diff)
downloadop-kernel-dev-df48fc74bffe09675b43aa2bd4b00c8cbb37e653.zip
op-kernel-dev-df48fc74bffe09675b43aa2bd4b00c8cbb37e653.tar.gz
usb: gadget: f_subset: fix missing unlock on error in geth_alloc()
Add the missing unlock before return from function geth_alloc() in the error handling case. Introduced by commit 02832e56f88a981474ee4c7c141f46fc1b4454f4. (usb: gadget: f_subset: add configfs support) Signed-off-by: Wei Yongjun <yongjun_wei@trendmicro.com.cn> Acked-by: Andrzej Pietrasiewicz <andrzej.p@samsung.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/usb/gadget')
-rw-r--r--drivers/usb/gadget/f_subset.c1
1 files changed, 1 insertions, 0 deletions
diff --git a/drivers/usb/gadget/f_subset.c b/drivers/usb/gadget/f_subset.c
index fbc7a24..5601e1d 100644
--- a/drivers/usb/gadget/f_subset.c
+++ b/drivers/usb/gadget/f_subset.c
@@ -548,6 +548,7 @@ static struct usb_function *geth_alloc(struct usb_function_instance *fi)
sizeof(geth->ethaddr));
if (status < 12) {
kfree(geth);
+ mutex_unlock(&opts->lock);
return ERR_PTR(-EINVAL);
}
geth_string_defs[1].s = geth->ethaddr;
OpenPOWER on IntegriCloud