Module Name: src
Committed By: matt
Date: Mon Jun 4 18:47:08 UTC 2012
Modified Files:
src/sys/kern [matt-nb5-mips64]: kern_sysctl.c
Log Message:
Each va_start needs a corresponding va_end, even in the error paths...
To generate a diff of this commit:
cvs rdiff -u -r1.218 -r1.218.8.1 src/sys/kern/kern_sysctl.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/kern/kern_sysctl.c
diff -u src/sys/kern/kern_sysctl.c:1.218 src/sys/kern/kern_sysctl.c:1.218.8.1
--- src/sys/kern/kern_sysctl.c:1.218 Thu Oct 23 20:41:14 2008
+++ src/sys/kern/kern_sysctl.c Mon Jun 4 18:47:07 2012
@@ -1,4 +1,4 @@
-/* $NetBSD: kern_sysctl.c,v 1.218 2008/10/23 20:41:14 christos Exp $ */
+/* kern_sysctl.c,v 1.218 2008/10/23 20:41:14 christos Exp */
/*-
* Copyright (c) 2003, 2007, 2008 The NetBSD Foundation, Inc.
@@ -68,7 +68,7 @@
*/
#include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: kern_sysctl.c,v 1.218 2008/10/23 20:41:14 christos Exp $");
+__KERNEL_RCSID(0, "kern_sysctl.c,v 1.218 2008/10/23 20:41:14 christos Exp");
#include "opt_defcorename.h"
#include "ksyms.h"
@@ -1935,19 +1935,26 @@ sysctl_createv(struct sysctllog **log, i
*/
va_start(ap, newlen);
namelen = 0;
+ error = 0;
ni = -1;
do {
- if (++ni == CTL_MAXNAME)
- return (ENAMETOOLONG);
+ if (++ni == CTL_MAXNAME) {
+ error = ENAMETOOLONG;
+ break;
+ }
name[ni] = va_arg(ap, int);
/*
* sorry, this is not supported from here
*/
- if (name[ni] == CTL_CREATESYM)
- return (EINVAL);
+ if (name[ni] == CTL_CREATESYM) {
+ error = EINVAL;
+ break;
+ }
} while (name[ni] != CTL_EOL && name[ni] != CTL_CREATE);
- namelen = ni + (name[ni] == CTL_CREATE ? 1 : 0);
va_end(ap);
+ if (error)
+ return error;
+ namelen = ni + (name[ni] == CTL_CREATE ? 1 : 0);
/*
* what's it called
