# HG changeset patch
# User Kim Alvefur <zash@zash.se>
# Date 1731973981 -3600
# Node ID d9965707f7dc24cf9012bd5064b5a0d2fe7e5115
# Parent  ca041359c0457107c44eb334b49ff7fa15ff404a
mod_smacks: Prevent traceback on attempting to resume destroyed session

This shouldn't be possible, but seems to happen anyway after net.server
somehow calling the close handler twice, which also should not happen.

diff -r ca041359c045 -r d9965707f7dc plugins/mod_smacks.lua
--- a/plugins/mod_smacks.lua	Tue Nov 19 00:41:02 2024 +0100
+++ b/plugins/mod_smacks.lua	Tue Nov 19 00:53:01 2024 +0100
@@ -587,6 +587,11 @@
 
 	local id = stanza.attr.previd;
 	local original_session = session_registry[registry_key(session, id)];
+	if original_session.destroyed then
+		original_session.log("error", "Tried to resume a destroyed session. This should not happen! %s", debug.traceback());
+		session_registry[registry_key(session, id)] = nil;
+		original_session = nil;
+	end
 	if not original_session then
 		local old_session = old_session_registry:get(session.username, id);
 		if old_session then