sin/samba-mirror
1
0
Fork 0
mirror of https://github.com/samba-team/samba.git synced 2025-01-27 14:04:05 +03:00
Code

Fix bug #8493 - DFS breaks zip file extracting unless "follow symlinks = no" set

Browse Source 
If a client sends a mangled name as part of a DFS path, use the
post-mangled name for the pathname walk, not the mangled name.

Autobuild-User: Jeremy Allison <jra@samba.org>
Autobuild-Date: Sat Oct  1 00:45:59 CEST 2011 on sn-devel-104
This commit is contained in:
Jeremy Allison 2011-09-30 13:35:59 -07:00
parent 5c569a97ab
commit c704d9216d
1 changed files with 2 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
source3/smbd/msdfs.c
View File

@ -577,15 +577,10 @@ static NTSTATUS dfs_path_lookup(TALLOC_CTX *ctx,
conn->connectpath, pdp->reqpath)); conn->connectpath, pdp->reqpath));
/* /*
* Note the unix path conversion here we're doing we can * Note the unix path conversion here we're doing we
* throw away. We're looking for a symlink for a dfs * throw away. We're looking for a symlink for a dfs
* resolution, if we don't find it we'll do another * resolution, if we don't find it we'll do another
* unix_convert later in the codepath. * unix_convert later in the codepath.
* If we needed to remember what we'd resolved in
* dp->reqpath (as the original code did) we'd
* copy (localhost, dp->reqpath) on any code
* path below that returns True - but I don't
* think this is needed. JRA.
*/ */
status = unix_convert(ctx, conn, pdp->reqpath, &smb_fname, status = unix_convert(ctx, conn, pdp->reqpath, &smb_fname,
@ -596,11 +591,7 @@ static NTSTATUS dfs_path_lookup(TALLOC_CTX *ctx,
NT_STATUS_OBJECT_PATH_NOT_FOUND)) { NT_STATUS_OBJECT_PATH_NOT_FOUND)) {
return status; return status;
} }
if (smb_fname == NULL || smb_fname->base_name == NULL) {
/* Create an smb_fname to use below. */
status = create_synthetic_smb_fname(ctx, pdp->reqpath, NULL,
NULL, &smb_fname);
if (!NT_STATUS_IS_OK(status)) {
return status; return status;
} }
} }