Subject: PR/36768 CVS commit: src/sys/netinet6
To: None <kern-bug-people@netbsd.org, gnats-admin@netbsd.org,>
From: DEGROOTE Arnaud <degroote@netbsd.org>
List: netbsd-bugs
Date: 09/22/2007 11:35:02
The following reply was made to PR kern/36768; it has been noted by GNATS.

From: DEGROOTE Arnaud <degroote@netbsd.org>
To: gnats-bugs@NetBSD.org
Cc: 
Subject: PR/36768 CVS commit: src/sys/netinet6
Date: Sat, 22 Sep 2007 11:32:07 +0000 (UTC)

 Module Name:	src
 Committed By:	degroote
 Date:		Sat Sep 22 11:32:07 UTC 2007
 
 Modified Files:
 	src/sys/netinet6: ah_output.c esp_output.c ipcomp_output.c
 
 Log Message:
 {ah,esp,ipcomp}_output must return 0 on success. On failure, it returns the
 error and m is freed. Previously, it was not the case in ipcomp and esp case
 (aka in some case, it returns 0 with m freed, or an error and m was not freed).
 
 In ipcomp_output, fix some leak of mcopy too.
 
 Use the same error path in {ah,esp,ipcomp}_output.
 
 Problem was reported by Wolfgang Stukenbrock in pr/36768.
 
 
 To generate a diff of this commit:
 cvs rdiff -r1.29 -r1.30 src/sys/netinet6/ah_output.c \
     src/sys/netinet6/esp_output.c
 cvs rdiff -r1.21 -r1.22 src/sys/netinet6/ipcomp_output.c
 
 Please note that diffs are not public domain; they are subject to the
 copyright notices on the relevant files.