Purging lint from the Bruce filter.

1 files changed, 10 insertions, 12 deletions

diff --git a/sys/kern/kern_sysctl.c b/sys/kern/kern_sysctl.c
index 958fa65..7418800 100644
--- a/sys/kern/kern_sysctl.c
+++ b/sys/kern/kern_sysctl.c
@@ -37,7 +37,7 @@
  * SUCH DAMAGE.
  *
  *	@(#)kern_sysctl.c	8.4 (Berkeley) 4/14/94
- * $Id: kern_sysctl.c,v 1.84 1999/02/16 10:49:48 dfr Exp $
+ * $Id: kern_sysctl.c,v 1.85 1999/03/23 14:23:15 phk Exp $
  */
 
 #include "opt_compat.h"
@@ -408,6 +408,8 @@ sysctl_sysctl_name2oid SYSCTL_HANDLER_ARGS
 
 	if (!req->newlen) 
 		return ENOENT;
+	if (req->newlen >= MAXPATHLEN)	/* XXX arbitrary, undocumented */
+		return (ENAMETOOLONG);
 
 	p = malloc(req->newlen+1, M_SYSCTL, M_WAITOK);
 
@@ -511,10 +513,7 @@ sysctl_handle_int SYSCTL_HANDLER_ARGS
 }
 
 /*
- * Handle a long, signed or unsigned.
- * Two cases:
- *     a variable:  point arg1 at it.
- *     a constant:  pass it in arg2.
+ * Handle a long, signed or unsigned.  arg1 points to it.
  */
 
 int
@@ -522,15 +521,14 @@ sysctl_handle_long SYSCTL_HANDLER_ARGS
 {
 	int error = 0;
 
+	if (!arg1)
+		return (EINVAL);
 	error = SYSCTL_OUT(req, arg1, sizeof(long));
 
 	if (error || !req->newptr)
 		return (error);
 
-	if (!arg1)
-		error = EPERM;
-	else
-		error = SYSCTL_IN(req, arg1, sizeof(long));
+	error = SYSCTL_IN(req, arg1, sizeof(long));
 	return (error);
 }
 
@@ -548,11 +546,11 @@ sysctl_handle_string SYSCTL_HANDLER_ARGS
 
 	error = SYSCTL_OUT(req, arg1, strlen((char *)arg1)+1);
 
-	if (error || !req->newptr || !arg2)
+	if (error || !req->newptr)
 		return (error);
 
-	if ((req->newlen - req->newidx) > arg2) {
-		error = E2BIG;
+	if ((req->newlen - req->newidx) >= arg2) {
+		error = EINVAL;
 	} else {
 		arg2 = (req->newlen - req->newidx);
 		error = SYSCTL_IN(req, arg1, arg2);