From a9ccd50b63421187c18bfb04001d225d212cdc2a Mon Sep 17 00:00:00 2001 From: Christian Ambach Date: Tue, 5 Apr 2016 02:58:48 +0200 Subject: [PATCH] s3:smbd/filename remove smelly code not sure how this chunk ended up there, but I agree with the statement in the comment that behavior should not depend on developer mode make test does not seem to depend on it anymore. This piece had some bad influence on the tests I wrote for case insensitivite behavior of SMB2/3, so let us remove this technical debt. Signed-off-by: Christian Ambach Reviewed-by: Jeremy Allison --- source3/smbd/filename.c | 28 ---------------------------- 1 file changed, 28 deletions(-) diff --git a/source3/smbd/filename.c b/source3/smbd/filename.c index 89c8bd63c35..7062060bcf4 100644 --- a/source3/smbd/filename.c +++ b/source3/smbd/filename.c @@ -942,34 +942,6 @@ NTSTATUS unix_convert(TALLOC_CTX *ctx, TALLOC_FREE(found_name); } /* end else */ -#ifdef DEVELOPER - /* - * This sucks! - * We should never provide different behaviors - * depending on DEVELOPER!!! - */ - if (VALID_STAT(smb_fname->st)) { - bool delete_pending; - uint32_t name_hash; - - status = file_name_hash(conn, - smb_fname_str_dbg(smb_fname), - &name_hash); - if (!NT_STATUS_IS_OK(status)) { - goto fail; - } - - get_file_infos(vfs_file_id_from_sbuf(conn, - &smb_fname->st), - name_hash, - &delete_pending, NULL); - if (delete_pending) { - status = NT_STATUS_DELETE_PENDING; - goto fail; - } - } -#endif - /* * Add to the dirpath that we have resolved so far. */ -- 2.34.1