All of lore.kernel.org
 help / color / mirror / Atom feed
From: Marek Lindner <mareklindner@neomailbox.ch>
To: b.a.t.m.a.n@lists.open-mesh.org
Cc: Antonio Quartulli <antonio@open-mesh.com>,
	Antonio Quartulli <antonio@meshcoding.com>
Subject: Re: [B.A.T.M.A.N.] [PATCHv2 maint] batman-adv: fix local TT check for outgoing arp requests in DAT
Date: Mon, 31 Mar 2014 17:42:26 +0800	[thread overview]
Message-ID: <1834287.qPYedGCF74@diderot> (raw)
In-Reply-To: <1396110458-8608-1-git-send-email-antonio@meshcoding.com>

[-- Attachment #1: Type: text/plain, Size: 1073 bytes --]

On Saturday 29 March 2014 17:27:38 Antonio Quartulli wrote:
> From: Antonio Quartulli <antonio@open-mesh.com>
> 
> Change introduced by d6bd8b36fa1f3d72a6fd5942a6e9bde6ddafcd0d
> ("batman-adv: make DAT drop ARP requests targeting local clients")
> implements a check that prevents DAT from using the caching
> mechanism when the client that is supposed to provide a reply
> to an arp request is local.
> 
> However change brought by 3e26722bc9f248ec4316749fc1957365c0fa5e4b
> ("batman-adv: make the Distributed ARP Table vlan aware")
> has not converted the above check into its vlan aware version
> thus making it useless when the local client is behind a vlan.
> 
> Fix the behaviour by properly specifying the vlan when
> checking for a client being local or not.
> 
> Reported-by: Simon Wunderlich <simon@open-mesh.com>
> Signed-off-by: Antonio Quartulli <antonio@open-mesh.com>
> ---
> 
> Changes from v2:
> - added reported-by clause
> 
>  distributed-arp-table.c | 3 +--
>  1 file changed, 1 insertion(+), 2 deletions(-)

Applied in revision 1c2e700.

Thanks,
Marek

[-- Attachment #2: This is a digitally signed message part. --]
[-- Type: application/pgp-signature, Size: 490 bytes --]

      reply	other threads:[~2014-03-31  9:42 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-03-29 16:27 [B.A.T.M.A.N.] [PATCHv2 maint] batman-adv: fix local TT check for outgoing arp requests in DAT Antonio Quartulli
2014-03-31  9:42 ` Marek Lindner [this message]

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=1834287.qPYedGCF74@diderot \
    --to=mareklindner@neomailbox.ch \
    --cc=antonio@meshcoding.com \
    --cc=antonio@open-mesh.com \
    --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 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.