From mboxrd@z Thu Jan 1 00:00:00 1970 From: catab@embedromix.ro Subject: Re: [PATCH v5] IPv6 NAT: Do not drop DNATed 6to4/6rd packets Date: Mon, 30 Sep 2013 06:05:45 +0300 (EEST) Message-ID: References: <1379963485.3575.53.camel@joe-AO722> <1379966659-28838-1-git-send-email-catab@embedromix.ro> <20130924213606.GB4446@order.stressinduktion.org> <20130928.155750.1130089685321379918.davem@davemloft.net> Mime-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII; format=flowed Cc: hannes@stressinduktion.org, netdev@vger.kernel.org, yoshfuji@linux-ipv6.org, joe@perches.com To: David Miller Return-path: Received: from mail.embedromix.ro ([5.9.41.220]:39248 "EHLO mail.embedromix.ro" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753872Ab3I3DGK (ORCPT ); Sun, 29 Sep 2013 23:06:10 -0400 In-Reply-To: <20130928.155750.1130089685321379918.davem@davemloft.net> Sender: netdev-owner@vger.kernel.org List-ID: On Sat, 28 Sep 2013, David Miller wrote: > From: Hannes Frederic Sowa > Date: Tue, 24 Sep 2013 23:36:06 +0200 > >> On Mon, Sep 23, 2013 at 11:04:19PM +0300, Catalin(ux) M. BOIE wrote: >>> When a router is doing DNAT for 6to4/6rd packets the latest anti-spoofing >>> patch (218774dc) will drop them because the IPv6 address embedded >>> does not match the IPv4 destination. This patch will allow them to >>> pass by testing if we have an address that matches on 6to4/6rd interface. >>> I have been hit by this problem using Fedora and IPV6TO4_IPV4ADDR. >>> Also, log the dropped packets (with rate limit). >>> >>> Signed-off-by: Catalin(ux) M. BOIE >> >> Acked-by: Hannes Frederic Sowa > > Applied, but Catalin please strictly refer to changes in the following > precise format: > > commit $SHA1_ID ("Commit message header line text") > > Because SHA1_IDs are ambiguous, especially when the change in question > is backported into various -stable branches. > > The only way to resolve the ambiguity is to provide the commit message > text (in parenthesis and double quotes). Roger. Should I resubmit it? Should I send it also to 'stable'? Thank you. -- Catalin(ux) M. BOIE http://kernel.embedromix.ro/