CVS commit: src/sys/secmodel/extensions

[Martin Husemann]

Module Name:    src
Committed By:   martin
Date:           Thu Feb 28 15:23:25 UTC 2013

Modified Files:
        src/sys/secmodel/extensions: secmodel_extensions.c

Log Message:
Make the callback deal with embryonic connections which do not have
credentials yet. Fixes PR kern/47598.


To generate a diff of this commit:
cvs rdiff -u -r1.4 -r1.5 src/sys/secmodel/extensions/secmodel_extensions.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/secmodel/extensions/secmodel_extensions.c
diff -u src/sys/secmodel/extensions/secmodel_extensions.c:1.4 src/sys/secmodel/extensions/secmodel_extensions.c:1.5
--- src/sys/secmodel/extensions/secmodel_extensions.c:1.4	Mon Jan 28 00:51:29 2013
+++ src/sys/secmodel/extensions/secmodel_extensions.c	Thu Feb 28 15:23:24 2013
@@ -1,4 +1,4 @@
-/* $NetBSD: secmodel_extensions.c,v 1.4 2013/01/28 00:51:29 jym Exp $ */
+/* $NetBSD: secmodel_extensions.c,v 1.5 2013/02/28 15:23:24 martin Exp $ */
 /*-
  * Copyright (c) 2011 Elad Efrat <e...@netbsd.org>
  * All rights reserved.
@@ -27,7 +27,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: secmodel_extensions.c,v 1.4 2013/01/28 00:51:29 jym Exp $");
+__KERNEL_RCSID(0, "$NetBSD: secmodel_extensions.c,v 1.5 2013/02/28 15:23:24 martin Exp $");
 
 #include <sys/types.h>
 #include <sys/param.h>
@@ -481,6 +481,9 @@ secmodel_extensions_network_cb(kauth_cre
 	if (curtain != 0) {
 		struct socket *so = (struct socket *)arg1;
 
+		if (__predict_false(so == NULL || so->so_cred == NULL))
+			return KAUTH_RESULT_DENY;
+
 		if (!kauth_cred_uidmatch(cred, so->so_cred)) {
 			int error;
 			bool isroot = false;