fixed buffer overflow, sizeof() is a compiletime method yet buf's siz…

…e is determined during runtime (taken from thieso2's fork of proxytunnel)
dagwieers · Feb 3, 2016 · 5472415 · 5472415
1 parent a3a1ffa
commit 5472415
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/io.c b/io.c
@@ -55,8 +55,8 @@ int readline(PTSTREAM *pts) {
 
 	if( args_info.verbose_flag ) {
 		/* Copy line of data into dstr without trailing newline */
-		char *dstr = malloc(sizeof(buf) + 1);
-		strlcpy( dstr, buf, strlen(buf) - 1);
+		char *dstr = malloc(strlen(buf) + 1);
+		strlcpy( dstr, buf, strlen(buf) );
 		if (strcmp(dstr, ""))
 			message( " <- %s\n", dstr );
 	}