From 41c5277f57eac320efb99f22b8336b13d88e8e26 Mon Sep 17 00:00:00 2001 From: Jeremy Allison Date: Tue, 29 Apr 2014 16:59:55 -0700 Subject: 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 Reviewed-by: Volker Lendecke --- source3/smbd/reply.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) (limited to 'source3') diff --git a/source3/smbd/reply.c b/source3/smbd/reply.c index da59ca75ae..f737d7499a 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++; -- cgit