# HG changeset patch
# User Waqas Hussain <waqas20@gmail.com>
# Date 1238360271 -18000
# Node ID db022135aed44b757c71b184490dc313ce454166
# Parent  c63f9bc45a85afdacf76913ebe9049dc3e460f6d
Fixed: stanza_router: Respond with correct stanza error on malformed stanzas

diff -r c63f9bc45a85 -r db022135aed4 core/stanza_router.lua
--- a/core/stanza_router.lua	Mon Mar 30 01:55:56 2009 +0500
+++ b/core/stanza_router.lua	Mon Mar 30 01:57:51 2009 +0500
@@ -51,9 +51,11 @@
 
 	if not stanza.attr.xmlns then stanza.attr.xmlns = "jabber:client"; end -- FIXME Hack. This should be removed when we fix namespace handling.
 	-- TODO verify validity of stanza (as well as JID validity)
-	if stanza.name == "iq" and #stanza.tags > 1 then
-		if stanza.attr.type == "set" or stanza.attr.type == "get" then
-			error("Invalid IQ");
+	if stanza.attr.xmlns == "error" and #stanza.tags == 0 then return; end -- TODO invalid stanza, log
+	if stanza.name == "iq" then
+		if (stanza.attr.type == "set" or stanza.attr.type == "get") and #stanza.tags ~= 1 then
+			origin.send(st.error_reply(stanza, "modify", "bad-request"));
+			return;
 		end
 	end