Revert bogus fix, make a note that the var is meant to 'leak'

--HG--
extra : convert_revision : e0efd546c3cdb2828971e41af2afe43e850c641d

1 files changed, 2 insertions, 3 deletions

diff --git a/cli-runopts.c b/cli-runopts.c
index d8eada7..269f5b3 100644
--- a/cli-runopts.c
+++ b/cli-runopts.c
@@ -348,7 +348,8 @@ static void addforward(char* origstr, struct TCPFwdList** fwdlist) {
 
 	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 @@ static void addforward(char* origstr, struct TCPFwdList** fwdlist) {
 	newfwd->next = *fwdlist;
 	*fwdlist = newfwd;
 
-	m_free(str);
-
 	TRACE(("leave addforward: done"))
 	return;