diff options
author | Andrew Tridgell <tridge@samba.org> | 2001-07-04 04:17:58 +0000 |
---|---|---|
committer | Andrew Tridgell <tridge@samba.org> | 2001-07-04 04:17:58 +0000 |
commit | 59db9bcbf084a23c1f8c7e1c7017f064b8c1e59c (patch) | |
tree | 35b5eb8877410c2af9c46430881bbc63c9f94d9b | |
parent | d972dc06a189fc433c822d1e5eee650a02a79550 (diff) | |
download | samba-59db9bcbf084a23c1f8c7e1c7017f064b8c1e59c.tar.gz samba-59db9bcbf084a23c1f8c7e1c7017f064b8c1e59c.tar.bz2 samba-59db9bcbf084a23c1f8c7e1c7017f064b8c1e59c.zip |
check for bad usernames early in session setup
(This used to be commit 657836599a847578096696af27cd7c9f0d52c931)
-rw-r--r-- | source3/smbd/reply.c | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/source3/smbd/reply.c b/source3/smbd/reply.c index d27f7842eb..edcc3f4838 100644 --- a/source3/smbd/reply.c +++ b/source3/smbd/reply.c @@ -858,6 +858,12 @@ int reply_sesssetup_and_X(connection_struct *conn, char *inbuf,char *outbuf,int domain,native_os,native_lanman)); } + /* don't allow for weird usernames */ + alpha_strcpy(user, user, ". _-", sizeof(user)); + if (strstr(user, "..")) { + return bad_password_error(inbuf, outbuf); + } + DEBUG(3,("sesssetupX:name=[%s]\n",user)); /* If name ends in $ then I think it's asking about whether a */ |