Ticket #1911 (closed defect: fixed)

Opened 3 years ago

Last modified 22 months ago

SASL auth multielement token handling error

Reported by: tomek@… Owned by: asterix
Priority: high Milestone: 0.11.1
Component: xmpppy Version:
Severity: critical Keywords:
Cc: OS:

Description

There is a bug in handling multielement tokens in SASL auth. They simply aren't handled at all.

You can find a patch for this here: http://staff.xiaoka.com/smoku/stuff/Jabber/Gajim/gajim-0.9.1-SASL-auth.patch

Attachments

Change History

Changed 3 years ago by dkirov

  • status changed from new to closed
  • resolution set to invalid

We fixed it yesterday in svn. Gajim-0.10 will be released these days.

Thanks anyway.

Changed 2 years ago by Buy air bed

  • summary changed from SASL auth multielement token handling error to Buy air bed
  • milestone set to 0.11

Your site is good. Want air bed? Buy air bed

7f6563fdccb3c2c

Changed 22 months ago by smoku

  • status changed from closed to reopened
  • resolution invalid deleted
  • severity changed from major to critical

The implementation is still faulty.

Consider this challenge:

, realm="localhost", nonce="LlBV2txnO8RbB5hgs3KgiQ==", qop="auth, auth-int, ", , , charset=utf-8, algorithm=md5-sess,

According to http://www.ietf.org/rfc/rfc2831.txt it's valid.

But Gajim implementation fails it: (src/common/xmpp/auth_nb.py:182)

if chal.has_key('qop') and ((type(chal['qop']) == str and \
chal['qop'] =='auth') or (type(chal['qop']) == list and 'auth' in \
chal['qop'])):

This code does not recognise "auth, auth-int, " as str neither as list.

Changed 22 months ago by anonymous

  • milestone set to 0.11.1

Changed 22 months ago by smoku

src/common/xmpp/auth_nb.py needs to be changed in the following lines in the challenge_splitter() function:

line 52:

elif char == ',' or char == ' ' or char == '\t':

line 62:

elif char == "," or char == ' ' or char == '\t':

Changed 22 months ago by asterix

  • status changed from reopened to closed
  • resolution set to fixed

(In [7938]) [smoku] fix SASL auth. fixes #1911

Add/Change #1911 (SASL auth multielement token handling error)

Author



Change Properties
<Author field>
Action
as closed
Next status will be 'reopened'
 
Note: See TracTickets for help on using tickets.