linux-bluetooth.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Marcel Holtmann <marcel@holtmann.org>
To: Johan Hedberg <johan.hedberg@gmail.com>
Cc: David Herrmann <dh.herrmann@googlemail.com>,
	linux-bluetooth@vger.kernel.org,
	"Gustavo F. Padovan" <padovan@profusion.mobi>
Subject: Re: Binary PIN Support
Date: Wed, 08 Jun 2011 07:08:42 +0200	[thread overview]
Message-ID: <1307509728.2589.19.camel@aeonflux> (raw)
In-Reply-To: <20110608013644.GA8315@dell.ccr.corp.intel.com>

Hi Johan,

> > > Since the idea with hex-encoding binary PINs and prefixing them with
> > > '$' did not get accepted, I have some other ideas for this:
> > > 
> > > 1: Use hard coded PINs for those devices. That is, use VID/PID
> > > detection during pairing and send hardcoded PIN (or based on src/dst
> > > address). Problem: VID/PID information is not available during
> > > pairing or may not be available, yet. Some devices may even reject
> > > SDP requests when not paired (as mentioned on IRC). Maybe someone
> > > has an idea how to get this working.
> > 
> > if I remember this correctly, then Wiimote does support DeviceID profile
> > and thus reports proper VID/PID combinations.
> > 
> > So just create some auto-pair support in bluetoothd (if needed via
> > plugins) and just auto-pair them like we would do with Secure Simple
> > Pairing anyway.
> > 
> > There is really no need at all to involve the UI code and mess up the
> > D-Bus API as it is. It was never designed for hardcoded PIN codes other
> > then numbers and strings anyway.
> > 
> > > 2: Extend DBus Agent API with a function RequestBinaryPinCode()
> > > which works exactly like RequestPinCode() but returns a bytearray
> > > (with length) instead of a string. A dbus agent then needs to
> > > signal to bluetoothd that it provides this method and bluetoothd
> > > shall call RequestBinaryPinCode() instead of RequestPinCode().
> > > However, I couldn't figure out a proper way to signal this capability
> > > to bluetoothd.
> > > The agent could send some kind of EnabledBinaryPinSupport-Signal to
> > > bluetoothd...
> > 
> > No way ever. We had that problem when we initially had to support a
> > fallback for the agent in 3.x series. Never ever again.
> > 
> > > 3: Use some kind of encoding in the UTF-8 string as proposed with the
> > > '$' character.
> > > However, this might break old software.
> > 
> > And how is that going to work out in reality. Same as $ is a valid
> > character. So also a clear no.
> > 
> > Go with 1) and try to just auto-pair the Wiimotes. That is the right
> > approach and makes a lot sense. It might be the one with the most work
> > on getting the DeviceID support integrated properly, but it is the right
> > way forward here.
> 
> Does the Wiimote support DeviceID info through EIR? If not we'd need to
> change the way CreatePairedDevice works since right now SDP is done only
> *after* pairing.

the Wiimote is still a Bluetooth 2.0 (or even 1.2) device. So no EIR
available. And yes, you are right, we need to figure out on how we can
do DeviceID before pairing. Especially with Security Mode 3 this will
get funny.

Regards

Marcel



  reply	other threads:[~2011-06-08  5:08 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-06-01 14:47 Binary PIN Support David Herrmann
2011-06-01 15:32 ` Anderson Lizardo
2011-06-01 20:06   ` Gustavo F. Padovan
2011-06-02  1:08 ` Marcel Holtmann
2011-06-08  1:36   ` Johan Hedberg
2011-06-08  5:08     ` Marcel Holtmann [this message]
2011-06-16 16:18     ` David Herrmann
2011-06-16 20:27       ` Marcel Holtmann

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=1307509728.2589.19.camel@aeonflux \
    --to=marcel@holtmann.org \
    --cc=dh.herrmann@googlemail.com \
    --cc=johan.hedberg@gmail.com \
    --cc=linux-bluetooth@vger.kernel.org \
    --cc=padovan@profusion.mobi \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).