sin/samba-mirror
1
0
Fork 0
mirror of https://github.com/samba-team/samba.git synced 2025-08-03 04:22:09 +03:00
Code Activity

r21226: Fix bug #4377 (rename of "foo" -> "Foo" fails).

Browse Source 
This is actually an interesting case as it exposed bad code in our DFS redirect
module (that was where the bug was introduced).
Caused by our turning on dfsroot be default.
Jeremy.
(This used to be commit 5fe25588f5)
This commit is contained in:
Jeremy Allison
2007-02-07 22:20:31 +00:00
committed by Gerald (Jerry) Carter
parent 4901d7f10b
commit ffe646b1e6
1 changed files with 29 additions and 11 deletions
Download Patch File Download Diff File Expand all files Collapse all files

40
source3/smbd/msdfs.c
View File

@ -261,7 +261,7 @@ static BOOL parse_symlink(TALLOC_CTX *ctx, char *buf, struct referral **preflist
talloc CTX can be NULL here if reflistp and refcnt pointers are null.
**********************************************************************/
BOOL is_msdfs_link(TALLOC_CTX *ctx, connection_struct *conn, char *path,
BOOL is_msdfs_link(TALLOC_CTX *ctx, connection_struct *conn, const char *path,
struct referral **reflistp, int *refcnt,
SMB_STRUCT_STAT *sbufp)
{
@ -317,11 +317,15 @@ TALLOC_CTX can be NULL here if struct referral **reflistpp, int *refcntp
are also NULL.
*****************************************************************/
static BOOL resolve_dfs_path(TALLOC_CTX *ctx, const char *dfspath,
struct dfs_path *dp,
connection_struct *conn, BOOL search_flag,
struct referral **reflistpp, int *refcntp,
BOOL *self_referralp, int *consumedcntp)
static BOOL resolve_dfs_path(TALLOC_CTX *ctx,
const char *dfspath,
struct dfs_path *dp,
connection_struct *conn,
BOOL search_flag,
struct referral **reflistpp,
int *refcntp,
BOOL *self_referralp,
int *consumedcntp)
{
pstring localpath;
int consumed_level = 1;
@ -349,12 +353,25 @@ static BOOL resolve_dfs_path(TALLOC_CTX *ctx, const char *dfspath,
DEBUG(10,("resolve_dfs_path: Conn path = %s req_path = %s\n", conn->connectpath, dp->reqpath));
status = unix_convert(conn, dp->reqpath, False, NULL, &sbuf);
/* Should we terminate on status != NT_STATUS_OK ???? */
/*
* Note the unix path conversion here we're doing we can
* throw away. We're looking for a symlink for a dfs
* resolution, if we don't find it we'll do another
* 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
* pstrcpy(localhost, dp->reqpath) on any code
* path below that returns True - but I don't
* think this is needed. JRA.
*/
/* JRA... should we strlower the last component here.... ? */
pstrcpy(localpath, dp->reqpath);
status = unix_convert(conn, localpath, False, NULL, &sbuf);
if (!NT_STATUS_IS_OK(status)) {
return False;
}
/* check if need to redirect */
if (is_msdfs_link(ctx, conn, localpath, reflistpp, refcntp, NULL)) {
if ( search_flag ) {
@ -371,7 +388,7 @@ static BOOL resolve_dfs_path(TALLOC_CTX *ctx, const char *dfspath,
}
/* redirect if any component in the path is a link */
pstrcpy(reqpath, dp->reqpath);
pstrcpy(reqpath, localpath);
p = strrchr_m(reqpath, '/');
while (p) {
*p = '\0';
@ -448,9 +465,10 @@ BOOL dfs_redirect( pstring pathname, connection_struct *conn, BOOL search_wcard_
return True;
} else {
DEBUG(3,("dfs_redirect: Not redirecting %s.\n", pathname));
/* Form non-dfs tcon-relative path */
pstrcpy(pathname, dp.reqpath);
DEBUG(3,("dfs_redirect: Path converted to non-dfs path %s\n", pathname));
return False;
}