Mercurial > hg > freeDiameter
changeset 386:ce8d20725308
Allow empty User-Name RADIUS attributes
| author | Sebastien Decugis <sdecugis@nict.go.jp> |
|---|---|
| date | Tue, 06 Jul 2010 13:27:11 +0900 |
| parents | 69057a6d68ec |
| children | 743195485eec |
| files | extensions/app_radgw/rgwx_auth.c libfreeDiameter/messages.c |
| diffstat | 2 files changed, 4 insertions(+), 6 deletions(-) [+] |
line wrap: on
line diff
--- a/extensions/app_radgw/rgwx_auth.c Mon Jul 05 18:45:07 2010 +0900 +++ b/extensions/app_radgw/rgwx_auth.c Tue Jul 06 13:27:11 2010 +0900 @@ -375,11 +375,9 @@ break; 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; }
--- a/libfreeDiameter/messages.c Mon Jul 05 18:45:07 2010 +0900 +++ b/libfreeDiameter/messages.c Tue Jul 06 13:27:11 2010 +0900 @@ -1138,7 +1138,7 @@ memcpy(&avp->avp_storage, value, sizeof(union avp_value)); /* 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; memcpy(avp->avp_storage.os.data, value->os.data, value->os.len);