changeset 411:253958302381

Revert bogus fix, make a note that the var is meant to 'leak'
author Matt Johnston <matt@ucc.asn.au>
date Sun, 04 Feb 2007 10:44:07 +0000
parents ea0929224294
children 47bcc3536bd5
files cli-runopts.c
diffstat 1 files changed, 2 insertions(+), 3 deletions(-) [+]
line wrap: on
line diff
--- a/cli-runopts.c	Sat Feb 03 13:50:47 2007 +0000
+++ b/cli-runopts.c	Sun Feb 04 10:44:07 2007 +0000
@@ -348,7 +348,8 @@
 
 	TRACE(("enter addforward"))
 
-	/* We probably don't want to be editing argvs */
+	/* We need to split the original argument up. This var
+	   is never free()d. */ 
 	str = m_strdup(origstr);
 
 	listenport = str;
@@ -400,8 +401,6 @@
 	newfwd->next = *fwdlist;
 	*fwdlist = newfwd;
 
-	m_free(str);
-
 	TRACE(("leave addforward: done"))
 	return;