From f831c48bc8724c5084879f21b0b57f6a38166aa6 Mon Sep 17 00:00:00 2001 From: Jeremy Allison Date: Tue, 29 Apr 2014 16:59:55 -0700 Subject: [PATCH] s3: smbd : Fix wildcard unlink to fail if we get an error rather than trying to continue. This can break smbd if we end up leaving a SHARING_VIOLATION retry record on the queue. Signed-off-by: Jeremy Allison --- source3/smbd/reply.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/source3/smbd/reply.c b/source3/smbd/reply.c index da59ca7..f737d74 100644 --- a/source3/smbd/reply.c +++ b/source3/smbd/reply.c @@ -2909,9 +2909,10 @@ NTSTATUS unlink_internals(connection_struct *conn, struct smb_request *req, status = do_unlink(conn, req, smb_fname, dirtype); if (!NT_STATUS_IS_OK(status)) { + TALLOC_FREE(dir_hnd); TALLOC_FREE(frame); TALLOC_FREE(talloced); - continue; + goto out; } count++; -- 1.9.1.423.g4596e3a