Fix trying to call hook removal callback on unloaded plugin

When a plugin which registered an entity vtable hook with a remove callback is unloaded OnMapEnd, dhooks still tried to call the removal callback on the next frame after the map ended and all entities were destroyed.

Remove the plugin's hooks from the removal list when a plugin is unloaded. The hooks are free'd anyways.

Drifter321#3

extension.cpp

@@ -123,12 +123,12 @@ bool DHooks::SDK_OnMetamodLoad(ISmmAPI *ismm, char *error, size_t maxlength, boo
 
 void DHooks::OnPluginUnloaded(IPlugin *plugin)
 {
-	CleanupHooks(plugin->GetBaseContext());
-	RemoveAllCallbacksForContext(plugin->GetBaseContext());
 	if(g_pEntityListener)
 	{
 		g_pEntityListener->CleanupListeners(plugin->GetBaseContext());
 	}
+	CleanupHooks(plugin->GetBaseContext());
+	RemoveAllCallbacksForContext(plugin->GetBaseContext());
 }
 // The next 3 functions handle cleanup if our interfaces are going to be unloaded
 bool DHooks::QueryRunning(char *error, size_t maxlength)

listeners.cpp

@@ -38,6 +38,14 @@ void DHooksEntityListener::CleanupListeners(IPluginContext *pContext)
 			g_EntityListeners.remove(i);
 		}
 	}
+	for (int i = g_pRemoveList.length() -1; i >= 0; i--)
+	{
+		DHooksManager *manager = g_pRemoveList.at(i);
+		if (pContext == NULL || pContext == manager->callback->plugin_callback->GetParentRuntime()->GetDefaultContext())
+		{
+			g_pRemoveList.remove(i);
+		}
+	}
 }
 
 void DHooksEntityListener::OnEntityCreated(CBaseEntity *pEntity, const char *classname)