All of lore.kernel.org
 help / color / mirror / Atom feed
From: Daniel Wagner <wagi@monom.org>
To: Szymon Janc <szymon.janc@tieto.com>
Cc: Marcel Holtmann <marcel@holtmann.org>,
	"linux-bluetooth@vger.kernel.org"
	<linux-bluetooth@vger.kernel.org>,
	Peter Hurley <peter@hurleysoftware.com>
Subject: Re: rfcomm & encryption
Date: Wed, 04 Jan 2012 11:25:02 +0100	[thread overview]
Message-ID: <4F0428FE.4000300@monom.org> (raw)
In-Reply-To: <201201040910.02057.szymon.janc@tieto.com>

Hi Szymon,

On 04.01.2012 09:10, Szymon Janc wrote:
> Hi,
> 
>> so we are establishing the connection with security level of SDP only
>> hence no encryption required. Which is the only exception to run without
>> encryption when using SSP.
>>
>> Since we do not disconnected in between SDP and RFCOMM channels, we have
>> to do a security level upgrade here. And for some reason that gets
>> triggered, but does not force encryption to be switched on.
>>
>> With SSP enabled you should always switch on encryption when getting
>> authentication complete event. Actually generally speaking you should
>> always switch on encryption after authentication. Otherwise the
>> authentication is rather pointless anyway.
>>
>> Look at commit d7556e20, then this code got re-ordered. It does not look
>> correct to me anymore. We might need to redo the whole auth and encrypt
>> callback handling.
> 
> Some time ago there was a patch from Peter Hurley that should fixed that issue.
> I've just noticed that for some reason it was not merged upstream..
> (we use it in our own branch for some time already)
> 
> [PATCH v3] Bluetooth: Fix l2cap conn failures for ssp devices
> http://www.spinics.net/lists/linux-bluetooth/msg15312.html

Thanks for the pointer. This patch fixes the problem.

cheers,
daniel

  reply	other threads:[~2012-01-04 10:25 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-01-03 20:20 rfcomm & encryption Daniel Wagner
2012-01-03 20:34 ` Marcel Holtmann
2012-01-04  8:10   ` Szymon Janc
2012-01-04 10:25     ` Daniel Wagner [this message]
2012-01-04 10:29     ` Luiz Augusto von Dentz

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4F0428FE.4000300@monom.org \
    --to=wagi@monom.org \
    --cc=linux-bluetooth@vger.kernel.org \
    --cc=marcel@holtmann.org \
    --cc=peter@hurleysoftware.com \
    --cc=szymon.janc@tieto.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.