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 96283da..fc86424 100644 (file)
--- 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.