diff options
author | Andrew Tridgell <tridge@samba.org> | 2000-04-22 08:28:22 +0000 |
---|---|---|
committer | Andrew Tridgell <tridge@samba.org> | 2000-04-22 08:28:22 +0000 |
commit | f7608e853596bcb6e8d3d4dd6bb4407fa090fbc1 (patch) | |
tree | ef08f29251f5db48e50d83a744e1636080012064 | |
parent | d384e1801d7cd841f67f3c9216900d2e59e9d1d5 (diff) | |
download | samba-f7608e853596bcb6e8d3d4dd6bb4407fa090fbc1.tar.gz samba-f7608e853596bcb6e8d3d4dd6bb4407fa090fbc1.tar.bz2 samba-f7608e853596bcb6e8d3d4dd6bb4407fa090fbc1.zip |
fixed overlapping strcpy() found by insure
(This used to be commit 1106fa7f24d229c3877263b7a7dde359556435e6)
-rw-r--r-- | source3/smbd/reply.c | 15 |
1 files changed, 6 insertions, 9 deletions
diff --git a/source3/smbd/reply.c b/source3/smbd/reply.c index 3acfd988d6..baff8f2ac8 100644 --- a/source3/smbd/reply.c +++ b/source3/smbd/reply.c @@ -260,10 +260,9 @@ int reply_tcon(connection_struct *conn, /**************************************************************************** Reply to a tcon and X. ****************************************************************************/ - int reply_tcon_and_X(connection_struct *conn, char *inbuf,char *outbuf,int length,int bufsize) { - pstring service; + fstring service; pstring user; pstring password; pstring devicename; @@ -295,11 +294,9 @@ int reply_tcon_and_X(connection_struct *conn, char *inbuf,char *outbuf,int lengt passlen = strlen(password); } - fstrcpy(service,path+2); - p = strchr(service,'\\'); + p = strchr(path+2,'\\'); if (!p) return(ERROR(ERRSRV,ERRinvnetname)); - *p = 0; fstrcpy(service,p+1); p = strchr(service,'%'); if (p) { @@ -309,11 +306,11 @@ int reply_tcon_and_X(connection_struct *conn, char *inbuf,char *outbuf,int lengt StrnCpy(devicename,path + strlen(path) + 1,6); DEBUG(4,("Got device type %s\n",devicename)); - /* - * Ensure the user and password names are in UNIX codepage format. - */ + /* + * Ensure the user and password names are in UNIX codepage format. + */ - dos_to_unix(user,True); + dos_to_unix(user,True); if (!doencrypt) dos_to_unix(password,True); |