From: Alan T. DeKok <aland@freeradius.org>
Date: Wed, 2 Dec 2009 07:51:27 +0000 (+0100)
Subject: Fixed string copying in sub variable.
X-Git-Tag: release_2_1_8~53
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=b0b9e5ee7896ff820ac983e4a9606b95f40717ad;p=thirdparty%2Ffreeradius-server.git

Fixed string copying in sub variable.

Apparently the only reason this worked before is that no one used it.
The decode_attribute() function did the string copying itself,
and therefore avoided this.
---

diff --git a/src/main/util.c b/src/main/util.c
index 7bd0c7cece5..a18add7b799 100644
--- a/src/main/util.c
+++ b/src/main/util.c
@@ -550,6 +550,7 @@ int rad_copy_variable(char *to, const char *from)
 			if (sublen < 0) return sublen;
 			from += sublen;
 			to += sublen;
+			length += sublen;
 			break;
 
 		case '}':	/* end of variable expansion */