xfnw/plan9fox
1
0
Fork 0
Code Issues Pull requests Projects Releases Wiki Activity

Disallow '/' in file names.

Browse source 
A bad rename call could send a path with a '/' to cwfs.
	This is invalid, and should be disallowed.
This commit is contained in:
Ori Bernstein 2018-09-25 01:02:31 -07:00
parent 2f076f946f
commit 72a840b31d
1 changed files with 3 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
sys/src/cmd/cwfs/sub.c
View file

@ -540,8 +540,8 @@ loop:
/* /*
* what are legal characters in a name? * what are legal characters in a name?
* only disallow control characters. * only disallow control characters.
* a) utf avoids control characters. * utf avoids control characters, so we
* b) '/' may not be the separator * only need to inspect the ascii range.
*/ */
int int
checkname(char *n) checkname(char *n)
@ -556,7 +556,7 @@ checkname(char *n)
c = n[i] & 0xff; c = n[i] & 0xff;
if(c == 0) if(c == 0)
return 0; return 0;
if(c < 040) if(c < 040 || c == '/')
return Ename; return Ename;
} }
return Etoolong; return Etoolong;