linux-bluetooth.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: bugzilla-daemon@bugzilla.kernel.org
To: linux-bluetooth@vger.kernel.org
Subject: [Bug 106691] New: BNEP multicast filter is filtering multicast addresses in default case although differently specified
Date: Tue, 27 Oct 2015 11:32:01 +0000	[thread overview]
Message-ID: <bug-106691-62941@https.bugzilla.kernel.org/> (raw)

https://bugzilla.kernel.org/show_bug.cgi?id=106691

            Bug ID: 106691
           Summary: BNEP multicast filter is filtering multicast addresses
                    in default case although differently specified
           Product: Drivers
           Version: 2.5
    Kernel Version: 4.0.4
          Hardware: All
                OS: Linux
              Tree: Mainline
            Status: NEW
          Severity: high
          Priority: P1
         Component: Bluetooth
          Assignee: linux-bluetooth@vger.kernel.org
          Reporter: danny.schweizer@proofnet.de
        Regression: No

A Linux PC is connected with another device over Bluetooth PAN using a BNEP
interface.

Whenever a packet is tried to be sent over the BNEP interface, the function
"bnep_net_xmit()" in "net/bluetooth/bnep/netdev.c" is called. This function
calls "bnep_net_mc_filter()", which checks (if the destination address is
multicast) if the address is set in a certain multicast filter (&s->mc_filter).
If it is not, then it is not sent out.

This filter is only changed in two other functions, found in
net/bluetooth/bnep/core.c": in "bnep_ctrl_set_mc_filter()", which is only
called if a message of type "BNEP_FILTER_MULTI_ADDR_SET" is received.
Otherwise, it is set in "bnep_add_connection()", where it is set to a default
value which only adds the broadcast address to the filter:

set_bit(bnep_mc_hash(dev->broadcast), (ulong *) &s->mc_filter);


To sum up, if the BNEP interface does not receive any message of type
"BNEP_FILTER_MULTI_ADDR_SET", it will not send out any messages with multicast
destination addresses except for broadcast.


However, in the BNEP specification (page 27 in
http://grouper.ieee.org/groups/802/15/Bluetooth/BNEP.pdf), it is said that per
default, all multicast addresses should not be filtered, i.e. the BNEP
interface should be able to send packets with any multicast destination
address.

It seems that the default case is wrong: the multicast filter should not block
almost all multicast addresses, but should not filter out any.

This leads to the problem that e.g. Neighbor Solicitation messages sent with
Bluetooth PAN over the BNEP interface to a multicast destination address other
than broadcast are blocked and not sent out.

-- 
You are receiving this mail because:
You are the assignee for the bug.

             reply	other threads:[~2015-10-27 11:32 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-10-27 11:32 bugzilla-daemon [this message]
2015-10-30 12:29 ` [Bug 106691] BNEP multicast filter is filtering multicast addresses in default case although differently specified bugzilla-daemon

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=bug-106691-62941@https.bugzilla.kernel.org/ \
    --to=bugzilla-daemon@bugzilla.kernel.org \
    --cc=linux-bluetooth@vger.kernel.org \
    /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).