summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJeremy Allison <jra@samba.org>2004-03-18 02:17:23 +0000
committerJeremy Allison <jra@samba.org>2004-03-18 02:17:23 +0000
commit7e799b3036cde20de79f03c8908967ff502bdf90 (patch)
tree26e38c3aa5b862b8bfd920dc60938f02978a066d
parent41ea0d35aeb984cbd3bc3baf96594ab6b67c70e2 (diff)
downloadsamba-7e799b3036cde20de79f03c8908967ff502bdf90.tar.gz
samba-7e799b3036cde20de79f03c8908967ff502bdf90.tar.bz2
samba-7e799b3036cde20de79f03c8908967ff502bdf90.zip
Added a varient of Volker's patch to fix the "self-referrals as anonymous user"
problem. Jeremy. (This used to be commit 74de9a61407e6c5a15fc7becca085a1d40199080)
-rw-r--r--source3/msdfs/msdfs.c68
1 files changed, 49 insertions, 19 deletions
diff --git a/source3/msdfs/msdfs.c b/source3/msdfs/msdfs.c
index b77e2111bc..2df5fcf4f5 100644
--- a/source3/msdfs/msdfs.c
+++ b/source3/msdfs/msdfs.c
@@ -151,10 +151,8 @@ static BOOL create_conn_struct( connection_struct *conn, int snum, char *path)
if (vfs_ChDir(conn,conn->connectpath) != 0) {
DEBUG(3,("create_conn_struct: Can't ChDir to new conn path %s. Error was %s\n",
conn->connectpath, strerror(errno) ));
-#if 0 /* JRATEST ? */
talloc_destroy( conn->mem_ctx );
return False;
-#endif
}
return True;
}
@@ -407,12 +405,40 @@ BOOL dfs_redirect(pstring pathname, connection_struct* conn,
}
/**********************************************************************
+ Return a self referral.
+**********************************************************************/
+
+static BOOL self_ref(char *pathname, struct junction_map *jucn,
+ int *consumedcntp, BOOL *self_referralp)
+{
+ struct referral *ref;
+
+ if (self_referralp != NULL)
+ *self_referralp = True;
+
+ jucn->referral_count = 1;
+ if((ref = (struct referral*) malloc(sizeof(struct referral))) == NULL) {
+ DEBUG(0,("self_ref: malloc failed for referral\n"));
+ return False;
+ }
+
+ pstrcpy(ref->alternate_path,pathname);
+ ref->proximity = 0;
+ ref->ttl = REFERRAL_TTL;
+ jucn->referral_list = ref;
+ if (consumedcntp)
+ *consumedcntp = strlen(pathname);
+
+ return True;
+}
+
+/**********************************************************************
Gets valid referrals for a dfs path and fills up the
junction_map structure
- **********************************************************************/
+**********************************************************************/
-BOOL get_referred_path(char *pathname, struct junction_map* jucn,
- int* consumedcntp, BOOL* self_referralp)
+BOOL get_referred_path(char *pathname, struct junction_map *jucn,
+ int *consumedcntp, BOOL *self_referralp)
{
struct dfs_path dp;
@@ -421,12 +447,13 @@ BOOL get_referred_path(char *pathname, struct junction_map* jucn,
pstring conn_path;
int snum;
BOOL ret = False;
-
BOOL self_referral = False;
if (!pathname || !jucn)
return False;
+ ZERO_STRUCT(conns);
+
if (self_referralp)
*self_referralp = False;
else
@@ -453,7 +480,18 @@ BOOL get_referred_path(char *pathname, struct junction_map* jucn,
if ((snum = find_service(jucn->service_name)) < 0)
return False;
}
-
+
+ /*
+ * Self referrals are tested with a anonymous IPC connection and
+ * a GET_DFS_REFERRAL call to \\server\share. (which means dp.reqpath[0] points
+ * to an empty string). create_conn_struct cd's into the directory and will
+ * fail if it cannot (as the anonymous user). Cope with this.
+ */
+
+ if (dp.reqpath[0] == '\0') {
+ return self_ref(pathname, jucn, consumedcntp, self_referralp);
+ }
+
pstrcpy(conn_path, lp_pathname(snum));
if (!create_conn_struct(conn, snum, conn_path))
return False;
@@ -496,24 +534,16 @@ BOOL get_referred_path(char *pathname, struct junction_map* jucn,
/* if self_referral, fill up the junction map */
if (*self_referralp) {
- struct referral* ref;
- jucn->referral_count = 1;
- if((ref = (struct referral*) malloc(sizeof(struct referral))) == NULL) {
- DEBUG(0,("malloc failed for referral\n"));
+ if (self_ref(pathname, jucn, consumedcntp, self_referralp) == False) {
goto out;
}
-
- pstrcpy(ref->alternate_path,pathname);
- ref->proximity = 0;
- ref->ttl = REFERRAL_TTL;
- jucn->referral_list = ref;
- if (consumedcntp)
- *consumedcntp = strlen(pathname);
}
ret = True;
+
out:
- talloc_destroy( conn->mem_ctx );
+ if (conn->mem_ctx)
+ talloc_destroy( conn->mem_ctx );
return ret;
}