no need to print the name

the module isn't going away until after it's data has been freed.

diff --git a/src/lib/server/dl_module.c b/src/lib/server/dl_module.c
index d9b36408af9d676249ea8f0bad780fbd3f4c9fcb..517d34822674f633d86cefc6a54605a98b996a22 100644 (file)
--- a/src/lib/server/dl_module.c
+++ b/src/lib/server/dl_module.c
@@ -283,7 +283,7 @@ static void dl_module_instance_data_alloc(dl_module_inst_t *dl_inst, dl_module_t
        if (!module->common->inst_type) {
                talloc_set_name(data, "%s_t", module->dl->name ? module->dl->name : "config");
        } else {
-               talloc_set_name(data, "%s", module->common->inst_type);
+               talloc_set_name_const(data, module->common->inst_type);
        }
        dl_inst->data = data;
 

diff --git a/src/lib/server/module.c b/src/lib/server/module.c
index c3c8c58a6c0aae2e7361db9b46c1f933d1b2cd89..0338591ce9f9b161e834d2fd8f0ea86d3bc5a2c1 100644 (file)
--- a/src/lib/server/module.c
+++ b/src/lib/server/module.c
@@ -1260,7 +1260,7 @@ int modules_thread_instantiate(TALLOC_CTX *ctx, fr_event_list_t *el)
                        if (!mi->module->thread_inst_type) {
                                talloc_set_name(ti->data, "rlm_%s_thread_t", mi->module->name);
                        } else {
-                               talloc_set_name(ti->data, "%s", mi->module->thread_inst_type);
+                               talloc_set_name_const(ti->data, mi->module->thread_inst_type);
                        }
                }