From owner-freebsd-pf@FreeBSD.ORG  Wed Mar 11 12:50:55 2015
Return-Path: <owner-freebsd-pf@FreeBSD.ORG>
Delivered-To: freebsd-pf@FreeBSD.org
Received: from mx1.freebsd.org (mx1.freebsd.org
 [IPv6:2001:1900:2254:206a::19:1])
 (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits))
 (No client certificate requested)
 by hub.freebsd.org (Postfix) with ESMTPS id 88EAFAE4;
 Wed, 11 Mar 2015 12:50:55 +0000 (UTC)
Received: from venus.codepro.be (venus.codepro.be [IPv6:2a01:4f8:162:1127::2])
 (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256
 bits))
 (Client CN "*.codepro.be", Issuer "Gandi Standard SSL CA" (verified OK))
 by mx1.freebsd.org (Postfix) with ESMTPS id 4A6D76E2;
 Wed, 11 Mar 2015 12:50:54 +0000 (UTC)
Received: from vega.codepro.be (unknown [172.16.1.3])
 by venus.codepro.be (Postfix) with ESMTP id 7519012206;
 Wed, 11 Mar 2015 13:50:50 +0100 (CET)
Received: by vega.codepro.be (Postfix, from userid 1001)
 id 6E5BE2E316; Wed, 11 Mar 2015 13:50:50 +0100 (CET)
Date: Wed, 11 Mar 2015 13:50:50 +0100
From: Kristof Provost <kristof@sigsegv.be>
To: "Andrey V. Elsukov" <bu7cher@yandex.ru>
Subject: Re: [PATCH] Fix panic with pf fastroute
Message-ID: <20150311125050.GR1975@vega.codepro.be>
References: <1426064691-1238-1-git-send-email-kristof@sigsegv.be>
 <20150311083916.GQ1975@vega.codepro.be> <55000FDF.10007@yandex.ru>
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
In-Reply-To: <55000FDF.10007@yandex.ru>
X-PGP-Fingerprint: E114 D9EA 909E D469 8F57  17A5 7D15 91C6 9EFA F286
X-Checked-By-NSA: Probably
User-Agent: Mutt/1.5.23 (2014-03-12)
Cc: freebsd-net@FreeBSD.org, freebsd-pf@FreeBSD.org
X-BeenThere: freebsd-pf@freebsd.org
X-Mailman-Version: 2.1.18-1
Precedence: list
List-Id: "Technical discussion and general questions about packet filter
 \(pf\)" <freebsd-pf.freebsd.org>
List-Unsubscribe: <http://lists.freebsd.org/mailman/options/freebsd-pf>,
 <mailto:freebsd-pf-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/freebsd-pf/>
List-Post: <mailto:freebsd-pf@freebsd.org>
List-Help: <mailto:freebsd-pf-request@freebsd.org?subject=help>
List-Subscribe: <http://lists.freebsd.org/mailman/listinfo/freebsd-pf>,
 <mailto:freebsd-pf-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Wed, 11 Mar 2015 12:50:55 -0000

On 2015-03-11 12:50:23 (+0300), Andrey V. Elsukov <bu7cher@yandex.ru> wrote:
> It looks like there are some code paths that do a copy of original mbuf.
> Are you sure this doesn't introduce mbuf leak?
> 
I'll check again in the morning when I'm less drunk and jet lagged, but
I'm pretty confident this is correct.
There are only two exit points from pf_route6(), this one only happens
in case of FASTROUTE, not DUPTO (which is the one that duplicates).

Regards.
Kristof