aboutsummaryrefslogtreecommitdiffstats
path: root/protocols/jabber/iq.c
diff options
context:
space:
mode:
authorWilmer van der Gaast <wilmer@gaast.net>2009-10-17 16:13:40 +0100
committerWilmer van der Gaast <wilmer@gaast.net>2009-10-17 16:13:40 +0100
commit74349eb5b77e2143289ef98201d03870e0d2366a (patch)
treee2cf767bc518e3751283b911f913a38980ecae1b /protocols/jabber/iq.c
parent57d842193767ddbb87531ce9865b23fbdb72eadb (diff)
If you can't find what you expect in an IQ packet, it doesn't mean it's
incorrect, so stop spamming. Ideally it should return a feature-not- implemented but I'll do that later.
Diffstat (limited to 'protocols/jabber/iq.c')
-rw-r--r--protocols/jabber/iq.c5
1 files changed, 3 insertions, 2 deletions
diff --git a/protocols/jabber/iq.c b/protocols/jabber/iq.c
index 38c5a5a9..875b5c81 100644
--- a/protocols/jabber/iq.c
+++ b/protocols/jabber/iq.c
@@ -50,10 +50,11 @@ xt_status jabber_pkt_iq( struct xt_node *node, gpointer data )
else if( strcmp( type, "get" ) == 0 )
{
if( !( ( c = xt_find_node( node->children, "query" ) ) ||
- ( c = xt_find_node( node->children, "ping" ) ) ) || /* O_o WHAT is wrong with just <query/> ????? */
+ ( c = xt_find_node( node->children, "ping" ) ) ) ||
!( s = xt_find_attr( c, "xmlns" ) ) )
{
- imcb_log( ic, "Warning: Received incomplete IQ-%s packet", type );
+ /* Sigh. Who decided to suddenly invent new elements
+ instead of just sticking with <query/>? */
return XT_HANDLED;
}