Skip site navigation (1)Skip section navigation (2)
Date:      Sat, 31 Aug 2019 04:26:59 +0000 (UTC)
From:      Cy Schubert <cy@FreeBSD.org>
To:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-stable@freebsd.org, svn-src-stable-12@freebsd.org
Subject:   svn commit: r351635 - in stable: 11/sys/contrib/ipfilter/netinet 12/sys/contrib/ipfilter/netinet
Message-ID:  <201908310426.x7V4Qx1m074143@repo.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: cy
Date: Sat Aug 31 04:26:59 2019
New Revision: 351635
URL: https://svnweb.freebsd.org/changeset/base/351635

Log:
  MFC r351562:
  
  Destroy the mutex in case of error.
  
  Obtained from:	NetBSD ip_nat.c r1.7

Modified:
  stable/12/sys/contrib/ipfilter/netinet/ip_nat.c
Directory Properties:
  stable/12/   (props changed)

Changes in other areas also in this revision:
Modified:
  stable/11/sys/contrib/ipfilter/netinet/ip_nat.c
Directory Properties:
  stable/11/   (props changed)

Modified: stable/12/sys/contrib/ipfilter/netinet/ip_nat.c
==============================================================================
--- stable/12/sys/contrib/ipfilter/netinet/ip_nat.c	Sat Aug 31 04:25:35 2019	(r351634)
+++ stable/12/sys/contrib/ipfilter/netinet/ip_nat.c	Sat Aug 31 04:26:59 2019	(r351635)
@@ -3078,7 +3078,7 @@ ipf_nat_newrdr(fin, nat, ni)
 /* creating a new NAT structure for a "RDR" rule (incoming NAT translation) */
 /* and (3) building that structure and putting it into the NAT table(s).    */
 /*                                                                          */
-/* NOTE: natsave should NOT be used top point back to an ipstate_t struct   */
+/* NOTE: natsave should NOT be used to point back to an ipstate_t struct    */
 /*       as it can result in memory being corrupted.                        */
 /* ------------------------------------------------------------------------ */
 nat_t *
@@ -3406,6 +3406,7 @@ ipf_nat_insert(softc, softn, nat)
 	u_int hv0, hv1;
 	u_int sp, dp;
 	ipnat_t *in;
+	int ret;
 
 	/*
 	 * Try and return an error as early as possible, so calculate the hash
@@ -3488,7 +3489,10 @@ ipf_nat_insert(softc, softn, nat)
 		nat->nat_mtu[1] = GETIFMTU_4(nat->nat_ifps[1]);
 	}
 
-	return ipf_nat_hashtab_add(softc, softn, nat);
+	ret = ipf_nat_hashtab_add(softc, softn, nat);
+	if (ret == -1)
+		MUTEX_DESTROY(&nat->nat_lock);
+	return ret;
 }
 
 



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201908310426.x7V4Qx1m074143>