]> git.ipfire.org Git - thirdparty/freeradius-server.git/commitdiff
Disable foreign attribute check
authorArran Cudbard-Bell <a.cudbardb@freeradius.org>
Fri, 4 Jan 2019 09:00:56 +0000 (17:00 +0800)
committerArran Cudbard-Bell <a.cudbardb@freeradius.org>
Fri, 4 Jan 2019 09:00:56 +0000 (17:00 +0800)
src/lib/server/tmpl.c

index 72529a21677b79efcb731c3861f65a236474355a..ffec8863ab19ebd0fc2e9f740d5e5a12a2b32f5d 100644 (file)
@@ -812,7 +812,21 @@ ssize_t tmpl_afrom_attr_substr(TALLOC_CTX *ctx, vp_tmpl_t **out, char const *nam
                /*
                 *      Check that the attribute we resolved was from an allowed dictionary
                 */
-               } else if ((rules->dict_def && (found_in != rules->dict_def))) {
+               }
+#if 0
+               else if ((rules->dict_def && (found_in != rules->dict_def))) {
+               /*
+                *      @fixme - We can't enforce this until we support nested attributes
+                *      where the change of attribute context gives us a new dictionary.
+                *
+                *      i.e.
+                *
+                *      My-Dhcp-In-RADIUS-Attribute.My-DHCP-Attribute
+                *      |                          ||_ DHCP attribute
+                *      |                          |_ Lookup inside linking attribute triggers dictionary change
+                *      |_ RADIUS attribute
+                */
+
                        if (!rules->allow_foreign) {
                                fr_strerror_printf("Only attributes from the %s protocol are allowed here",
                                                   fr_dict_root(rules->dict_def)->name);
@@ -820,6 +834,7 @@ ssize_t tmpl_afrom_attr_substr(TALLOC_CTX *ctx, vp_tmpl_t **out, char const *nam
                                goto error;
                        }
                }
+#endif
        }
 
        /*