From 786476b93a0377f202d4bc7b156dbc6cce6fcf59 Mon Sep 17 00:00:00 2001 From: ben Date: Tue, 7 Aug 2007 22:02:03 +0000 Subject: [PATCH] Correctly terminate nc target hostname when copying it. While we're here, use memcpy rather than strncpy when we've already worked out how much we're going to copy. git-svn-id: svn://svn.tartarus.org/sgt/putty@7685 cda61777-01e9-0310-a592-d414129be87e --- cmdline.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/cmdline.c b/cmdline.c index 5b294578..183797df 100644 --- a/cmdline.c +++ b/cmdline.c @@ -263,8 +263,8 @@ int cmdline_process_param(char *p, char *value, int need_save, Config *cfg) unsigned len = portp - host; if (len >= sizeof(cfg->ssh_nc_host)) len = sizeof(cfg->ssh_nc_host) - 1; - strncpy(cfg->ssh_nc_host, value, len); - cfg->ssh_nc_host[sizeof(cfg->ssh_nc_host) - 1] = '\0'; + memcpy(cfg->ssh_nc_host, value, len); + cfg->ssh_nc_host[len] = '\0'; cfg->ssh_nc_port = atoi(portp+1); } else { cmdline_error("-nc expects argument of form 'host:port'"); -- 2.11.0