svn commit: r302256 - head/sbin/natd

[Poul-Henning Kamp]

Author: phk
Date: Tue Jun 28 20:10:30 2016
New Revision: 302256
URL: https://svnweb.freebsd.org/changeset/base/302256

Log:
  Do not coredump if the packet is too long in the global (non-interface
  associated) instance.
  
  The result is that the packet is dropped without an indication
  that smaller MTU is advisable, which is not optimal, but better
  than a NULL pointer deref.
  
  Approved by:	re (glebius)

Modified:
  head/sbin/natd/natd.c

Modified: head/sbin/natd/natd.c
==============================================================================
--- head/sbin/natd/natd.c	Tue Jun 28 19:53:16 2016	(r302255)
+++ head/sbin/natd/natd.c	Tue Jun 28 20:10:30 2016	(r302256)
@@ -618,7 +618,7 @@ static void DoGlobal (int fd)
 	
 	if (wrote != bytes) {
 
-		if (errno == EMSGSIZE) {
+		if (errno == EMSGSIZE && mip != NULL) {
 
 			if (mip->ifMTU != -1)
 				SendNeedFragIcmp (icmpSock,