allocate instance data even if the module doesn't need it

which lets the rest of the distinguish virtual attributes from
typos in xlat functions

diff --git a/src/main/modules.c b/src/main/modules.c
index fd4334db96e213beee068d9dcac00e4c6ed4a978..9ccb3103ae0a129b258c2e8c84e92b033144a46f 100644 (file)
--- a/src/main/modules.c
+++ b/src/main/modules.c
@@ -571,10 +571,10 @@ static int module_conf_parse(module_instance_t *node, void **handle)
         *      If there is supposed to be instance data, allocate it now.
         *      Also parse the configuration data, if required.
         */
-       if (node->entry->module->inst_size) {
-               *handle = talloc_zero_array(node, uint8_t, node->entry->module->inst_size);
-               rad_assert(*handle);
+       *handle = talloc_zero_array(node, uint8_t, node->entry->module->inst_size);
+       rad_assert(*handle);
 
+       if (node->entry->module->inst_size) {
                talloc_set_name(*handle, "rlm_%s_t",
                                node->entry->module->name ? node->entry->module->name : "config");