Re: PR/53036 CVS commit: src/sys/dist/pf/net

[Alexander Nasonov <alnsn%yandex.ru@localhost>]

The following reply was made to PR kern/53036; it has been noted by GNATS.

From: Alexander Nasonov <alnsn%yandex.ru@localhost>
To: Christos Zoulas <christos%zoulas.com@localhost>
Cc: Alexander Nasonov <alnsn%yandex.ru@localhost>, gnats-bugs%NetBSD.org@localhost,
	kern-bug-people%netbsd.org@localhost, gnats-admin%netbsd.org@localhost,
	netbsd-bugs%netbsd.org@localhost, alnsn%NetBSD.org@localhost
Subject: Re: PR/53036 CVS commit: src/sys/dist/pf/net
Date: Tue, 20 Feb 2018 09:22:06 +0000

 Christos Zoulas wrote:
 > Ok, how about this then?
 > 
 > christos
 > Index: uipc_socket2.c
 > ===================================================================
 > RCS file: /cvsroot/src/sys/kern/uipc_socket2.c,v
 > retrieving revision 1.126
 > diff -u -u -r1.126 uipc_socket2.c
 > --- uipc_socket2.c	6 Jul 2017 17:42:39 -0000	1.126
 > +++ uipc_socket2.c	19 Feb 2018 22:49:37 -0000
 > @@ -356,6 +356,9 @@
 >  	}
 >  	KASSERT(solocked2(head, so));
 >  
 > +	so->so_cred = head->so_cred;
 > +	kauth_cred_hold(so->so_cred);
 > +
 >  	/*
 >  	 * Insert into the queue.  If ready, update the connection status
 >  	 * and wake up any waiters, e.g. processes blocking on accept().
 
 I assume this change isn't needed. My server is running fine with your
 	if (so == NULL | so->so_cred == NULL) return -1;
 change.
 
 -- 
 Alex