ucode: initialize module search path only once

uh_ucode_state_init is invoked once per configured ucode prefix
from ucode_plugin_init. The search path lives in the file-static
'config' struct, and uc_search_path_init() appends the default
paths to that struct's vector each time it is called. Running it
per prefix therefore accumulates duplicate entries in the module
search path (and leaks the duplicate strdup'd strings) once more
than one ucode handler is configured.

Hoist the call to ucode_plugin_init so the default search path is
populated exactly once before any VMs are initialized.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Signed-off-by: Hauke Mehrtens <hauke@hauke-m.de>

1 files changed, 2 insertions, 1 deletions

diff --git a/ucode.c b/ucode.c
index bbb9b00..d80a509 100644
--- a/ucode.c
+++ b/ucode.c
@@ -231,7 +231,6 @@ uh_ucode_state_init(struct ucode_prefix *ucode)
 	uc_value_t *v;
 	int exitcode;
 
-	uc_search_path_init(&config.module_search_path);
 	uc_vm_init(vm, &config);
 	uc_stdlib_load(uc_vm_scope_get(vm));
 
@@ -430,6 +429,8 @@ ucode_plugin_init(const struct uhttpd_ops *o, struct config *c)
 	ops = o;
 	_conf = c;
 
+	uc_search_path_init(&config.module_search_path);
+
 	list_for_each_entry(p, &conf.ucode_prefix, list)
 		uh_ucode_state_init(p);