From: Simon Wunderlich <sw@simonwunderlich.de>
To: b.a.t.m.a.n@lists.open-mesh.org
Cc: alessandro@mediaspot.net
Subject: [B.A.T.M.A.N.] [PATCH-maint 0/4] Some fixes for DAT/TT/Speedy join corner cases
Date: Fri, 21 Aug 2015 17:15:14 +0200 [thread overview]
Message-ID: <1440170118-10876-1-git-send-email-sw@simonwunderlich.de> (raw)
Upon debugging a network with dangling, bogus TT entries, I found a
couple of bugs for which I would like to propose fixes. The network showed
the following symptoms:
* DAT was enabled
* VLANs were used, but not every originator used the same VLANs
* I've found global entries assigned to originators which actually
never had the client in question connected to them. These false target
originators didn't even had the VLANs in use or bridged for which
the entry was done. This caused packets sent to be sent to these wrong
originators.
* The wrong entries also didn't get purged automatically, since they
didn't announce the VLAN in question through their TT TLVLs, and the
TT code didn't check for excess VLANs.
* Furthermore, the temp flag was removed too early from the TT entries
so that the purge function was not removing the entry after a timeout
as well.
* I've found that with DAT, the cached ARP replies may cause these
TT entries to be created on behalf of the answering host. This is wrong,
since the answering host (usually) doesn't actually has the client
connected.
Three of these four patches fix various issues connected with the problem
described above. The fourth one is merely a style fix which does not neccessarily
have to be adopted to maint.
We have not yet tested these patches in the network, and the bug also appears
only once in a couple of days. Therefore I'd like to ask to review these patches
thoroughly, and if you agree to the fixes we will apply them in this production
network.
Thanks,
Simon
Simon Wunderlich (4):
batman-adv: fix speedy join for DAT cache replies
batman-adv: avoid keeping false temporary entry
batman-adv: unify flags access style in tt global add
batman-adv: detect local excess vlans in TT request
net/batman-adv/routing.c | 17 +++++++++++++----
net/batman-adv/translation-table.c | 33 ++++++++++++++++++++++++++++++---
2 files changed, 43 insertions(+), 7 deletions(-)
--
2.5.0
next reply other threads:[~2015-08-21 15:15 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-08-21 15:15 Simon Wunderlich [this message]
2015-08-21 15:15 ` [B.A.T.M.A.N.] [PATCH-maint 1/4] batman-adv: fix speedy join for DAT cache replies Simon Wunderlich
2015-08-25 9:42 ` Antonio Quartulli
2015-08-25 15:24 ` Simon Wunderlich
2015-08-21 15:15 ` [B.A.T.M.A.N.] [PATCH-maint 2/4] batman-adv: avoid keeping false temporary entry Simon Wunderlich
2015-08-25 9:49 ` Antonio Quartulli
2015-08-25 15:27 ` Simon Wunderlich
2015-08-21 15:15 ` [B.A.T.M.A.N.] [PATCH-maint 3/4] batman-adv: unify flags access style in tt global add Simon Wunderlich
2015-08-25 9:51 ` Antonio Quartulli
2015-08-25 15:28 ` Simon Wunderlich
2015-08-25 16:14 ` Antonio Quartulli
2015-08-21 15:15 ` [B.A.T.M.A.N.] [PATCH-maint 4/4] batman-adv: detect local excess vlans in TT request Simon Wunderlich
2015-08-25 9:59 ` Antonio Quartulli
2015-08-25 15:31 ` Simon Wunderlich
2015-08-25 18:28 ` Antonio Quartulli
2015-08-25 21:12 ` Simon Wunderlich
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=1440170118-10876-1-git-send-email-sw@simonwunderlich.de \
--to=sw@simonwunderlich.de \
--cc=alessandro@mediaspot.net \
--cc=b.a.t.m.a.n@lists.open-mesh.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