Revert "Do not use new/delete to allocate modules, allows modules to always destruct properly and automatically"

This does not work as expected, it causes objects allocated by modules to be freed by the operating system when
the module is unloaded, giving no chance to the module to deallocate it itself.

This reverts commit 05e6815d912f0418f6da25a2106dd718796f02fa.

1 files changed, 16 insertions, 2 deletions

diff --git a/src/modulemanager.cpp b/src/modulemanager.cpp
index 8ce75a849..d2dd6cd53 100644
--- a/src/modulemanager.cpp
+++ b/src/modulemanager.cpp
@@ -255,8 +255,22 @@ void ModuleManager::DeleteModule(Module *m)
 	ano_module_t handle = m->handle;
 	Anope::string filename = m->filename;
 
-	if (handle && dlclose(handle))
-		Log() << ano_moderr();
+	ano_modclearerr();
+	void (*destroy_func)(Module *m) = function_cast<void (*)(Module *)>(dlsym(m->handle, "AnopeFini"));
+	const char *err = ano_moderr();
+	if (!destroy_func && err && *err)
+	{
+		Log() << "No destroy function found, chancing delete...";
+		delete m; /* we just have to chance they haven't overwrote the delete operator then... */
+	}
+	else
+		destroy_func(m); /* Let the module delete it self, just in case */
+
+	if (handle)
+	{
+		if (dlclose(handle))
+			Log() << ano_moderr();
+	}
 
 	if (!filename.empty())
 		DeleteFile(filename.c_str());