Subject: Re: PR/36119 CVS commit: src/sys/netipsec
To: None <kern-bug-people@netbsd.org, gnats-admin@netbsd.org,>
From: Karl Knutsson <karl.knutsson@ericsson.com>
List: netbsd-bugs
Date: 04/10/2007 08:55:02
The following reply was made to PR kern/36119; it has been noted by GNATS.

From: Karl Knutsson <karl.knutsson@ericsson.com>
To: gnats-bugs@NetBSD.org
Cc: kern-bug-people@netbsd.org, gnats-admin@netbsd.org,
	netbsd-bugs@netbsd.org
Subject: Re: PR/36119 CVS commit: src/sys/netipsec
Date: Tue, 10 Apr 2007 10:54:34 +0200

 Hi,
 
 On Mon, Apr 09, 2007 at 09:10:02PM +0000, DEGROOTE Arnaud wrote:
 > The following reply was made to PR kern/36119; it has been noted by GNATS.
 > 
 > From: DEGROOTE Arnaud <degroote@netbsd.org>
 > To: gnats-bugs@NetBSD.org
 > Cc: 
 > Subject: PR/36119 CVS commit: src/sys/netipsec
 > Date: Mon,  9 Apr 2007 21:07:03 +0000 (UTC)
 > 
 >  Module Name:	src
 >  Committed By:	degroote
 >  Date:		Mon Apr  9 21:07:03 UTC 2007
 >  
 >  Modified Files:
 >  	src/sys/netipsec: key.c
 >  
 >  Log Message:
 >  Fix a memleak in key_spdget.
 >  
 >  Problem was reported by Karl Knutsson by pr/36119.
 >  
 >  
 >  To generate a diff of this commit:
 >  cvs rdiff -r1.43 -r1.44 src/sys/netipsec/key.c
 >  
 >  Please note that diffs are not public domain; they are subject to the
 >  copyright notices on the relevant files.
 >  
 
 Thanks, that fix corrected the memleak. 
 
 The sequence number in the reply is still hard coded to zero. That might be
 a problem if you have an application that matches replies to requests based 
 on the pid and sequence number in the reply (as per the PFKEY RFC 2367).
 
 best regards
 Karl Knutsson