From: Alan T. DeKok <aland@freeradius.org>
Date: Sun, 22 Nov 2009 15:47:29 +0000 (+0100)
Subject: More warnings
X-Git-Tag: release_2_1_8~66
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=cbccacc520550af8c68c20769957d5ef16024389;p=thirdparty%2Ffreeradius-server.git

More warnings

Modifying the values of a virtual attribute is a bad idea.
---

diff --git a/src/main/evaluate.c b/src/main/evaluate.c
index 96283dad9bc..fc864241c31 100644
--- a/src/main/evaluate.c
+++ b/src/main/evaluate.c
@@ -1316,6 +1316,12 @@ int radius_update_attrlist(REQUEST *request, CONF_SECTION *cs,
 
 		cp = cf_itemtopair(ci);
 
+#ifndef NDEBUG
+		if (debug_flag && radius_find_compare(vp->attribute)) {
+			DEBUG("WARNING: You are modifying the value of virtual attribute %s.  This is not supported.", vp->name);
+		}
+#endif
+
 		/*
 		 *	The VP && CF lists should be in sync.  If they're
 		 *	not, panic.