IQ Reply Spoofing

From XMPP WIKI
Jump to navigation Jump to search

Information

The Situation

Most XMPP stacks provide a convenience method to send an IQ request that returns the IQ response (or e.g. throws an exception on timeout). To collect the response the incoming IQ stanzas are matched against a filter. Unfortunately the filter often looks like

StanzaFilter badFilter = AndFilter(StanzaTypeFilter(IQ), StanzaIdFilter(iqrequest.id))

Such a filter would therefore match every IQ stanza with the same ID as the request. But this is not enough. A malicious attacker that is able to guess the request IQ's ID is able to send a spoofed IQ response, which causes the malicious response to be evaluated by the XMPP stack. This gives the attacker the ability to trigger code parts (usually parsing related code) and logic he normally wouldn't have access to. For example, he could able to add contacts to the victims rooster.

The solution is to additionally verify the from attribute of the result IQ, since its value can not be spoofed:

StanzaFilter iqResponseFilter = AndFilter(badFilter, IQFromFilter(iqrequest.to))

Now the devil is in the details: It is valid to send an stanza, and this includes an IQ request, without a to attribute (RFC 6120 § 8.1.1.1). Furthermore some servers may reply with the users full JID if the request was send to the bare JID. IQFromFilter must therefore, besides matching stanzas that have exactly the same 'from' value as the requests 'to' attribute, follow those rules:

If 'to' is not set, match stanzas where 'from' is

  • not set
  • the sending entity's bare JID, when the resource part is stripped
  • the XMPP service JID

If 'to' is the sending entity's bare JID, the it must also match stanzas where 'to' is not set.

Links with more information

Software Components

Vulnerable

Not Vulnerable / Fixed