mirrors/coturn
1
0
Fork 0
mirror of https://github.com/coturn/coturn.git synced 2025-11-01 07:21:04 +01:00
Code Issues Projects Releases Wiki Activity

refactor is_secure_username

Browse Source
This commit is contained in:
Mészáros Mihály 2017-11-03 14:25:30 +01:00
parent 5ff0d8b4fa
commit bb821cee2e
4 changed files with 7 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
src/apps/common/ns_turn_utils.c
View File

@ -867,11 +867,11 @@ char *turn_strdup_func(const char* s, const char* function, int line) {
////////////////////////////////
int is_secure_username(const u08bits *username)
int is_secure_string(const u08bits *string)
{
int ret = 0;
if(username) {
unsigned char *s0 = (unsigned char*)turn_strdup((const char*)username);
if(string) {
unsigned char *s0 = (unsigned char*)turn_strdup((const char*)string);
unsigned char *s = s0;
while(*s) {
*s = (unsigned char)tolower((int)*s);
@ -891,4 +891,3 @@ int is_secure_username(const u08bits *username)
}
//////////////////////////////////////////////////////////////////

2
src/apps/common/ns_turn_utils.h
View File

@ -78,7 +78,7 @@ void rollover_logfile(void);
///////////////////////////////////////////////////////
int is_secure_username(const u08bits *username);
int is_secure_string(const u08bits *string);
///////////////////////////////////////////////////////

2
src/apps/relay/mainrelay.c
View File

@ -1790,7 +1790,7 @@ static int adminmain(int argc, char **argv)
#endif
case 'u':
STRCPY(user,optarg);
if(!is_secure_username((u08bits*)user)) {
if(!is_secure_string((u08bits*)user)) {
TURN_LOG_FUNC(TURN_LOG_LEVEL_ERROR, "Wrong user name structure or symbols, choose another name: %s\n",user);
exit(-1);
}

4
src/server/ns_turn_server.c
View File

@ -1029,7 +1029,7 @@ static int handle_turn_allocate(turn_turnserver *server,
}
ns_bcopy(value,username,ulen);
username[ulen]=0;
if(!is_secure_username(username)) {
if(!is_secure_string(username)) {
TURN_LOG_FUNC(TURN_LOG_LEVEL_ERROR, "%s: wrong username: %s\n", __FUNCTION__, (char*)username);
username[0]=0;
*err_code = 400;
@ -3383,7 +3383,7 @@ static int check_stun_auth(turn_turnserver *server,
ns_bcopy(stun_attr_get_value(sar),usname,alen);
usname[alen]=0;
if(!is_secure_username(usname)) {
if(!is_secure_string(usname)) {
TURN_LOG_FUNC(TURN_LOG_LEVEL_ERROR, "%s: wrong username: %s\n", __FUNCTION__, (char*)usname);
usname[0]=0;
*err_code = 400;