diff options
| author | Christian Ambach <ambi@samba.org> | 2016-04-05 02:58:48 +0200 |
|---|---|---|
| committer | Jeremy Allison <jra@samba.org> | 2016-05-04 01:28:23 +0200 |
| commit | a9ccd50b63421187c18bfb04001d225d212cdc2a (patch) | |
| tree | 5e0720ce65b2c0c6c84620e62cc59d36aa9faded /source3/smbd/filename.c | |
| parent | 32427768610cbe2e5dfe199ab7865b061ea57919 (diff) | |
| download | samba-a9ccd50b63421187c18bfb04001d225d212cdc2a.tar.gz | |
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 <ambi@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
Diffstat (limited to 'source3/smbd/filename.c')
| -rw-r--r-- | source3/smbd/filename.c | 28 |
1 files changed, 0 insertions, 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. */ |