> -----Original Message----- > From: Dan Carpenter [mailto:dan.carpenter@xxxxxxxxxx] > Sent: Monday, November 13, 2017 4:06 AM > To: Darren Hart <dvhart@xxxxxxxxxxxxx>; Limonciello, Mario > <Mario_Limonciello@xxxxxxxx> > Cc: Andy Shevchenko <andy@xxxxxxxxxxxxx>; platform-driver- > x86@xxxxxxxxxxxxxxx; kernel-janitors@xxxxxxxxxxxxxxx > Subject: [PATCH] platform/x86: wmi: Unlock on error in wmi_ioctl() > > We need to drop a lock if try_module_get() fails. > > Fixes: 44b6b7661132 ("platform/x86: wmi: create userspace interface for drivers") > Signed-off-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx> > > diff --git a/drivers/platform/x86/wmi.c b/drivers/platform/x86/wmi.c > index 8c31ed4f0e1b..791449a2370f 100644 > --- a/drivers/platform/x86/wmi.c > +++ b/drivers/platform/x86/wmi.c > @@ -868,8 +868,10 @@ static long wmi_ioctl(struct file *filp, unsigned int cmd, > unsigned long arg) > /* let the driver do any filtering and do the call */ > wdriver = container_of(wblock->dev.dev.driver, > struct wmi_driver, driver); > - if (!try_module_get(wdriver->driver.owner)) > - return -EBUSY; > + if (!try_module_get(wdriver->driver.owner)) { > + ret = -EBUSY; > + goto out_ioctl; > + } > ret = wdriver->filter_callback(&wblock->dev, cmd, buf); > module_put(wdriver->driver.owner); > if (ret) Hi Dan, Thanks for submitting. This exact same fix is in the subsystem "testing" branch but hasn't yet been promoted to -next. http://git.infradead.org/users/dvhart/linux-platform-drivers-x86.git/commit/5e3e22971fb0b0d7d03286f1a619d0348748a243
