summaryrefslogtreecommitdiff
path: root/source3/smbd
diff options
context:
space:
mode:
authorJeremy Allison <jra@samba.org>2004-02-25 01:35:14 +0000
committerJeremy Allison <jra@samba.org>2004-02-25 01:35:14 +0000
commit1d9f07735c67a29f4cf56aa376d7a1f034c967f6 (patch)
tree379ed1ed657271445e99c7c5f203b90947ae4e08 /source3/smbd
parent01cb68643c129729f3df2bdfcdc81bd3c7fef71a (diff)
downloadsamba-1d9f07735c67a29f4cf56aa376d7a1f034c967f6.tar.gz
samba-1d9f07735c67a29f4cf56aa376d7a1f034c967f6.tar.bz2
samba-1d9f07735c67a29f4cf56aa376d7a1f034c967f6.zip
More gentest fun :-). NB. I'm not fixing OpenX breakage 'cos if you look
at what W2K3 accepts here it's COMPLETELY BROKEN ! :-). Jeremy. (This used to be commit 5107f6ee70a9300678ccb7b3c812e23ddf795c22)
Diffstat (limited to 'source3/smbd')
-rw-r--r--source3/smbd/reply.c32
1 files changed, 27 insertions, 5 deletions
diff --git a/source3/smbd/reply.c b/source3/smbd/reply.c
index d0888fc682..c729f22728 100644
--- a/source3/smbd/reply.c
+++ b/source3/smbd/reply.c
@@ -1457,8 +1457,12 @@ NTSTATUS unlink_internals(connection_struct *conn, int dirtype, char *name)
pstring fname;
pstrcpy(fname,dname);
- if((strcmp(fname, ".") == 0) || (strcmp(fname, "..")==0))
- continue;
+ /* Quick check for "." and ".." */
+ if (fname[0] == '.') {
+ if (!fname[1] || (fname[1] == '.' && !fname[2])) {
+ continue;
+ }
+ }
if(!mask_match(fname, mask, case_sensitive))
continue;
@@ -3357,6 +3361,13 @@ NTSTATUS rename_internals_fsp(connection_struct *conn, files_struct *fsp, char *
ZERO_STRUCT(sbuf);
unix_convert(newname,conn,newname_last_component,&bad_path,&sbuf);
+ /* Quick check for "." and ".." */
+ if (newname_last_component[0] == '.') {
+ if (!newname_last_component[1] || (newname_last_component[1] == '.' && !newname_last_component[2])) {
+ return NT_STATUS_ACCESS_DENIED;
+ }
+ }
+
/* Ensure newname contains a '/' */
if(strrchr_m(newname,'/') == 0) {
pstring tmpstr;
@@ -3470,6 +3481,13 @@ NTSTATUS rename_internals(connection_struct *conn, char *name, char *newname, BO
rc = unix_convert(name,conn,0,&bad_path1,&sbuf1);
unix_convert(newname,conn,newname_last_component,&bad_path2,&sbuf2);
+ /* Quick check for "." and ".." */
+ if (newname_last_component[0] == '.') {
+ if (!newname_last_component[1] || (newname_last_component[1] == '.' && !newname_last_component[2])) {
+ return NT_STATUS_ACCESS_DENIED;
+ }
+ }
+
/*
* Split the old name into directory and last component
* strings. Note that unix_convert may have stripped off a
@@ -3478,7 +3496,7 @@ NTSTATUS rename_internals(connection_struct *conn, char *name, char *newname, BO
* name and newname contain a / character or neither of them do
* as this is checked in resolve_wildcards().
*/
-
+
p = strrchr_m(name,'/');
if (!p) {
pstrcpy(directory,".");
@@ -3657,8 +3675,12 @@ directory = %s, newname = %s, newname_last_component = %s, is_8_3 = %d\n",
pstrcpy(fname,dname);
- if((strcmp(fname, ".") == 0) || (strcmp(fname, "..")==0))
- continue;
+ /* Quick check for "." and ".." */
+ if (fname[0] == '.') {
+ if (!fname[1] || (fname[1] == '.' && !fname[2])) {
+ continue;
+ }
+ }
if(!mask_match(fname, mask, case_sensitive))
continue;