diff options
author | Jeremy Allison <jra@samba.org> | 2019-02-24 08:15:23 -0800 |
---|---|---|
committer | Karolin Seeger <kseeger@samba.org> | 2019-03-05 07:39:29 +0000 |
commit | 03e4555f344e6806c80fefc8cf00a08d6ea2766e (patch) | |
tree | 9541ac2f2c92e33ecebc22b31e1c035d620ac578 /source3 | |
parent | c249c88cdb731026da04819e165717bb89278a42 (diff) | |
download | samba-03e4555f344e6806c80fefc8cf00a08d6ea2766e.tar.gz |
s3: smbd: filenames - ensure we replace the missing '/' if we error in an intermediate POSIX path.
Previous regression test ensures we still return the correct
error code for POSIX pathname operations.
BUG: https://bugzilla.samba.org/show_bug.cgi?id=13803
Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Volker Lendecke <vl@samba.org>
Autobuild-User(master): Volker Lendecke <vl@samba.org>
Autobuild-Date(master): Mon Feb 25 09:33:27 CET 2019 on sn-devel-144
(cherry picked from commit 3f1a13a94a753c5cb3b9f2cf795df5adb0f74205)
Diffstat (limited to 'source3')
-rw-r--r-- | source3/smbd/filename.c | 18 |
1 files changed, 18 insertions, 0 deletions
diff --git a/source3/smbd/filename.c b/source3/smbd/filename.c index 4c8c5ea19bc..8a00fbb06e7 100644 --- a/source3/smbd/filename.c +++ b/source3/smbd/filename.c @@ -917,6 +917,24 @@ NTSTATUS unix_convert(TALLOC_CTX *ctx, * * BUG: https://bugzilla.samba.org/show_bug.cgi?id=13803 */ + if (end != NULL) { + const char *morepath = NULL; + /* + * If this is intermediate we must + * restore the full path. + */ + *end = '/'; + /* + * If there are any more components + * after the failed LSTAT we cannot + * continue. + */ + morepath = strchr(end + 1, '/'); + if (morepath != NULL) { + status = NT_STATUS_OBJECT_PATH_NOT_FOUND; + goto fail; + } + } if (errno == ENOENT) { /* New file or directory. */ goto done; |