Fix for bug #105.

	ALWAYS create a new Vendor-Specific attribute when trying to encode
	multiple VSA's.  Some older RADIUS implementations do not understand
	multiple VSA's in a Vendor-Specific.

diff --git a/src/lib/radius.c b/src/lib/radius.c
index 30e9fc9..6404117 100644 (file)
--- a/src/lib/radius.c
+++ b/src/lib/radius.c
@@ -155,9 +155,17 @@ int rad_send(RADIUS_PACKET *packet, const char *secret)
                          /*
                           *    If the Vendor-Specific attribute is getting
                           *    full, then create a new VSA attribute
+                          *
+                          *    FIXME: Multiple VSA's per Vendor-Specific
+                          *    SHOULD be configurable.  When that's done,
+                          *    the (1), below, can be changed to point to
+                          *    a configuration variable which is set TRUE
+                          *    if the NAS cannot understand multiple VSA's
+                          *    per Vendor-Specific
                           */
-                         if (vsa_length_ptr &&
-                             (reply->length + *vsa_length_ptr) >= MAX_STRING_LEN) {
+                         if ((1) || /* ALWAYS create a new Vendor-Specific */
+                             (vsa_length_ptr &&
+                              (reply->length + *vsa_length_ptr) >= MAX_STRING_LEN)) {
                                  vendorcode = 0;
                                  vendorpec = 0;
                                  vsa_length_ptr = NULL;