Module Name:    src
Committed By:   rmind
Date:           Wed May 14 20:35:27 UTC 2014

Modified Files:
        src/sys/net/npf: npf_session.c

Log Message:
npf_session_inspect: do not silently drop the packet on state check failure.
Let the rules deal with it (e.g. we may want to log it).


To generate a diff of this commit:
cvs rdiff -u -r1.31 -r1.32 src/sys/net/npf/npf_session.c

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.

Modified files:

Index: src/sys/net/npf/npf_session.c
diff -u src/sys/net/npf/npf_session.c:1.31 src/sys/net/npf/npf_session.c:1.32
--- src/sys/net/npf/npf_session.c:1.31	Fri Mar 14 11:29:44 2014
+++ src/sys/net/npf/npf_session.c	Wed May 14 20:35:27 2014
@@ -1,4 +1,4 @@
-/*	$NetBSD: npf_session.c,v 1.31 2014/03/14 11:29:44 rmind Exp $	*/
+/*	$NetBSD: npf_session.c,v 1.32 2014/05/14 20:35:27 rmind Exp $	*/
 
 /*-
  * Copyright (c) 2010-2013 The NetBSD Foundation, Inc.
@@ -92,7 +92,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: npf_session.c,v 1.31 2014/03/14 11:29:44 rmind Exp $");
+__KERNEL_RCSID(0, "$NetBSD: npf_session.c,v 1.32 2014/05/14 20:35:27 rmind Exp $");
 
 #include <sys/param.h>
 #include <sys/types.h>
@@ -590,10 +590,9 @@ npf_session_inspect(npf_cache_t *npc, nb
 
 	/* Inspect the protocol data and handle state changes. */
 	if (!npf_state_inspect(npc, nbuf, &se->s_state, forw)) {
-		/* Silently block invalid packets. */
+		/* Invalid: let the rules deal with it. */
 		npf_session_release(se);
 		npf_stats_inc(NPF_STAT_INVALID_STATE);
-		*error = ENETUNREACH;
 		se = NULL;
 	}
 	return se;

Reply via email to