Revert "Don't do tlv checks on 'internal' attributes". Fixes #1272
authorAlan T. DeKok <aland@freeradius.org>
Fri, 25 Sep 2015 11:08:28 +0000 (07:08 -0400)
committerAlan T. DeKok <aland@freeradius.org>
Fri, 25 Sep 2015 11:08:28 +0000 (07:08 -0400)
This reverts commit e040518aa15a29d3fbb6fafb764080997a9c9a71.

src/lib/dict.c

index a4ea0aa..8079f67 100644 (file)
@@ -777,7 +777,7 @@ int dict_addattr(char const *name, int attr, unsigned int vendor, PW_TYPE type,
                 *      Otherwise, dict_parent() has taken us from an Extended sub-attribute to
                 *      a *the* Extended attribute, whish isn't what we want here.
                 */
-               if (!flags.internal && (vendor == parent->vendor) && (parent->type != PW_TYPE_TLV)) {
+               if ((vendor == parent->vendor) && (parent->type != PW_TYPE_TLV)) {
                        fr_strerror_printf("dict_addattr: Attribute %s has parent attribute %s which is not of type 'tlv'",
                                           name, parent->name);
                        return -1;