* [PATCH 1/3] cifs: Delete a stray unlock in cifs_swn_reconnect()
@ 2020-12-17 11:01 Dan Carpenter
2020-12-17 11:40 ` Samuel Cabrero
0 siblings, 1 reply; 2+ messages in thread
From: Dan Carpenter @ 2020-12-17 11:01 UTC (permalink / raw)
To: Steve French
Cc: Aurelien Aptel, Samuel Cabrero, linux-cifs, samba-technical,
linux-kernel, kernel-janitors
The unlock is done in the caller, this is a stray which leads to a
double unlock bug.
Fixes: bf80e5d4259a ("cifs: Send witness register and unregister commands to userspace daemon")
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
---
fs/cifs/cifs_swn.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/fs/cifs/cifs_swn.c b/fs/cifs/cifs_swn.c
index c594e588a8b5..b2ef082d6438 100644
--- a/fs/cifs/cifs_swn.c
+++ b/fs/cifs/cifs_swn.c
@@ -285,8 +285,6 @@ static struct cifs_swn_reg *cifs_find_swn_reg(struct cifs_tcon *tcon)
continue;
}
- mutex_unlock(&cifs_swnreg_idr_mutex);
-
cifs_dbg(FYI, "Existing swn registration for %s:%s found\n", swnreg->net_name,
swnreg->share_name);
--
2.29.2
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [PATCH 1/3] cifs: Delete a stray unlock in cifs_swn_reconnect()
2020-12-17 11:01 [PATCH 1/3] cifs: Delete a stray unlock in cifs_swn_reconnect() Dan Carpenter
@ 2020-12-17 11:40 ` Samuel Cabrero
0 siblings, 0 replies; 2+ messages in thread
From: Samuel Cabrero @ 2020-12-17 11:40 UTC (permalink / raw)
To: Dan Carpenter, Steve French
Cc: Aurelien Aptel, linux-cifs, samba-technical, linux-kernel,
kernel-janitors
On Thu, 2020-12-17 at 14:01 +0300, Dan Carpenter wrote:
> The unlock is done in the caller, this is a stray which leads to a
> double unlock bug.
>
Indeed.
Reviewed-by: Samuel Cabrero <scabrero@suse.de>
--
Samuel Cabrero / SUSE Labs Samba Team
GPG: D7D6 E259 F91C F0B3 2E61 1239 3655 6EC9 7051 0856
scabrero@suse.com
scabrero@suse.de
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2020-12-17 11:41 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-12-17 11:01 [PATCH 1/3] cifs: Delete a stray unlock in cifs_swn_reconnect() Dan Carpenter
2020-12-17 11:40 ` Samuel Cabrero
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®