Skip site navigation (1)Skip section navigation (2) 
Date:      Fri, 11 Mar 2011 18:56:55 +0000 (UTC)
From:      Matthew D Fleming <mdf@FreeBSD.org>
To:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org
Subject:   svn commit: r219523 - in head/sys: amd64/amd64 ia64/ia64 powerpc/aim sparc64/sparc64 sun4v/sun4v sys
Message-ID:  <201103111856.p2BIut2E090250@svn.freebsd.org>
next in thread | raw e-mail | index | archive | help 
Author: mdf
Date: Fri Mar 11 18:56:55 2011
New Revision: 219523
URL: http://svn.freebsd.org/changeset/base/219523

Log:
  Mostly revert r219468, as I had misremembered the C standard regarding
  the size of an extern array.
  
  Keep one change from strncpy to strlcpy.

Modified:
  head/sys/amd64/amd64/machdep.c
  head/sys/ia64/ia64/machdep.c
  head/sys/powerpc/aim/machdep.c
  head/sys/sparc64/sparc64/machdep.c
  head/sys/sun4v/sun4v/machdep.c
  head/sys/sys/kernel.h

Modified: head/sys/amd64/amd64/machdep.c
==============================================================================
--- head/sys/amd64/amd64/machdep.c	Fri Mar 11 18:51:42 2011	(r219522)
+++ head/sys/amd64/amd64/machdep.c	Fri Mar 11 18:56:55 2011	(r219523)
@@ -1758,7 +1758,7 @@ hammer_time(u_int64_t modulep, u_int64_t
 
         env = getenv("kernelname");
 	if (env != NULL)
-		strlcpy(kernelname, env, MAXPATHLEN);
+		strlcpy(kernelname, env, sizeof(kernelname));
 
 #ifdef XENHVM
 	if (inw(0x10) == 0x49d2) {

Modified: head/sys/ia64/ia64/machdep.c
==============================================================================
--- head/sys/ia64/ia64/machdep.c	Fri Mar 11 18:51:42 2011	(r219522)
+++ head/sys/ia64/ia64/machdep.c	Fri Mar 11 18:56:55 2011	(r219523)
@@ -792,7 +792,7 @@ ia64_init(void)
 
 	p = getenv("kernelname");
 	if (p != NULL) {
-		strlcpy(kernelname, p, MAXPATHLEN);
+		strlcpy(kernelname, p, sizeof(kernelname));
 		freeenv(p);
 	}
 

Modified: head/sys/powerpc/aim/machdep.c
==============================================================================
--- head/sys/powerpc/aim/machdep.c	Fri Mar 11 18:51:42 2011	(r219522)
+++ head/sys/powerpc/aim/machdep.c	Fri Mar 11 18:56:55 2011	(r219523)
@@ -540,7 +540,7 @@ powerpc_init(vm_offset_t startkernel, vm
 	 */
         env = getenv("kernelname");
         if (env != NULL) {
-		strlcpy(kernelname, env, MAXPATHLEN);
+		strlcpy(kernelname, env, sizeof(kernelname));
 		freeenv(env);
 	}
 

Modified: head/sys/sparc64/sparc64/machdep.c
==============================================================================
--- head/sys/sparc64/sparc64/machdep.c	Fri Mar 11 18:51:42 2011	(r219522)
+++ head/sys/sparc64/sparc64/machdep.c	Fri Mar 11 18:56:55 2011	(r219523)
@@ -532,7 +532,7 @@ sparc64_init(caddr_t mdp, u_long o1, u_l
 	init_param2(physmem);
 	env = getenv("kernelname");
 	if (env != NULL) {
-		strlcpy(kernelname, env, MAXPATHLEN);
+		strlcpy(kernelname, env, sizeof(kernelname));
 		freeenv(env);
 	}
 

Modified: head/sys/sun4v/sun4v/machdep.c
==============================================================================
--- head/sys/sun4v/sun4v/machdep.c	Fri Mar 11 18:51:42 2011	(r219522)
+++ head/sys/sun4v/sun4v/machdep.c	Fri Mar 11 18:56:55 2011	(r219523)
@@ -425,7 +425,7 @@ sparc64_init(caddr_t mdp, u_long o1, u_l
 
 	env = getenv("kernelname");
 	if (env != NULL) {
-		strlcpy(kernelname, env, MAXPATHLEN);
+		strlcpy(kernelname, env, sizeof(kernelname));
 		freeenv(env);
 	}
 

Modified: head/sys/sys/kernel.h
==============================================================================
--- head/sys/sys/kernel.h	Fri Mar 11 18:51:42 2011	(r219522)
+++ head/sys/sys/kernel.h	Fri Mar 11 18:56:55 2011	(r219523)
@@ -55,7 +55,7 @@
 /* Global variables for the kernel. */
 
 /* 1.1 */
-extern char kernelname[/*MAXPATHLEN*/];
+extern char kernelname[MAXPATHLEN];
 
 extern int tick;			/* usec per tick (1000000 / hz) */
 extern int hz;				/* system clock's frequency */

Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201103111856.p2BIut2E090250>