diff options
| author | Wilmer van der Gaast <wilmer@gaast.net> | 2009-10-17 16:13:40 +0100 |
|---|---|---|
| committer | Wilmer van der Gaast <wilmer@gaast.net> | 2009-10-17 16:13:40 +0100 |
| commit | 74349eb5b77e2143289ef98201d03870e0d2366a (patch) | |
| tree | e2cf767bc518e3751283b911f913a38980ecae1b | |
| parent | 57d842193767ddbb87531ce9865b23fbdb72eadb (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.
| -rw-r--r-- | protocols/jabber/iq.c | 5 |
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; } |