Changeset 386:ce8d20725308 in freeDiameter

Timestamp:

Jul 6, 2010, 1:27:11 PM (14 years ago)

Author:

Sebastien Decugis <sdecugis@nict.go.jp>

Branch:

default

Phase:

public

Message:

Allow empty User-Name RADIUS attributes

Files:

• extensions/app_radgw/rgwx_auth.c (modified) (1 diff)
• libfreeDiameter/messages.c (modified) (1 diff)

extensions/app_radgw/rgwx_auth.c

@@ -376 +376 @@
                 
                         case RADIUS_ATTR_USER_NAME:
-                                if (attr_len) {
-                                        TRACE_DEBUG(ANNOYING, "Found a User-Name attribute: '%.*s'", attr_len, attr_val);
-                                        un = attr_val;
-                                        un_len = attr_len;
-                                }
+                                TRACE_DEBUG(ANNOYING, "Found a User-Name attribute: '%.*s'", attr_len, attr_val);
+                                un = attr_val;
+                                un_len = attr_len;
                                 break;
                         

libfreeDiameter/messages.c

@@ -1139 +1139 @@
         
         /* Copy an octetstring if needed. */
-        if (type == AVP_TYPE_OCTETSTRING) {
+        if ((type == AVP_TYPE_OCTETSTRING) && (value->os.len)) {
                 CHECK_MALLOC(  avp->avp_storage.os.data = malloc(value->os.len)  );
                 avp->avp_mustfreeos = 1;