nhyatt/putty-source
1
0
Fork 0
mirror of https://git.tartarus.org/simon/putty.git synced 2025-01-10 01:48:00 +00:00
Code Issues Projects Releases Wiki Activity

Correctly terminate nc target hostname when copying it.

Browse Source 
While we're here, use memcpy rather than strncpy when we've already worked
out how much we're going to copy.

[originally from svn r7685]
This commit is contained in:
Ben Harris 2007-08-07 22:02:03 +00:00
parent ac041a3d66
commit be8818b4bc
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
cmdline.c
View File

@ -263,8 +263,8 @@ int cmdline_process_param(char *p, char *value, int need_save, Config *cfg)
unsigned len = portp - host; unsigned len = portp - host;
if (len >= sizeof(cfg->ssh_nc_host)) if (len >= sizeof(cfg->ssh_nc_host))
len = sizeof(cfg->ssh_nc_host) - 1; len = sizeof(cfg->ssh_nc_host) - 1;
strncpy(cfg->ssh_nc_host, value, len); memcpy(cfg->ssh_nc_host, value, len);
cfg->ssh_nc_host[sizeof(cfg->ssh_nc_host) - 1] = '\0'; cfg->ssh_nc_host[len] = '\0';
cfg->ssh_nc_port = atoi(portp+1); cfg->ssh_nc_port = atoi(portp+1);
} else { } else {
cmdline_error("-nc expects argument of form 'host:port'"); cmdline_error("-nc expects argument of form 'host:port'");