svn commit: r334201 - stable/11/sys/contrib/ipfilter/netinet

[Cy Schubert]

Author: cy
Date: Fri May 25 06:25:33 2018
New Revision: 334201
URL: https://svnweb.freebsd.org/changeset/base/334201

Log:
  MFC r333392-r333393, r333427
  
  r333392:
  Fix memory leak. (CID 1199373).
  
  r333393:
  Document intentional fallthrough. (CID 976535)
  
  r333427:
  Fix style error introduced in r333393.
  
  Reported by:	jhb, imp, phk
  
  Approved by:	re (delphij)

Modified:
  stable/11/sys/contrib/ipfilter/netinet/fil.c
  stable/11/sys/contrib/ipfilter/netinet/ip_dstlist.c
Directory Properties:
  stable/11/   (props changed)

Modified: stable/11/sys/contrib/ipfilter/netinet/fil.c
==============================================================================
--- stable/11/sys/contrib/ipfilter/netinet/fil.c	Fri May 25 03:34:33 2018	(r334200)
+++ stable/11/sys/contrib/ipfilter/netinet/fil.c	Fri May 25 06:25:33 2018	(r334201)
@@ -1299,6 +1299,7 @@ ipf_pr_icmp(fin)
 			}
 		}
 #endif
+		/* FALLTHROUGH */
 	case ICMP_SOURCEQUENCH :
 	case ICMP_REDIRECT :
 	case ICMP_TIMXCEED :

Modified: stable/11/sys/contrib/ipfilter/netinet/ip_dstlist.c
==============================================================================
--- stable/11/sys/contrib/ipfilter/netinet/ip_dstlist.c	Fri May 25 03:34:33 2018	(r334200)
+++ stable/11/sys/contrib/ipfilter/netinet/ip_dstlist.c	Fri May 25 06:25:33 2018	(r334201)
@@ -690,6 +690,7 @@ ipf_dstlist_node_del(softc, arg, op, uid)
 	err = COPYIN(op->iplo_struct, temp, size);
 	if (err != 0) {
 		IPFERROR(120027);
+		KFREES(temp, size);
 		return EFAULT;
 	}