Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 26 Feb 2009 10:14:10 +0000 (UTC)
From:      Ed Schouten <ed@FreeBSD.org>
To:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org
Subject:   svn commit: r189060 - head/sys/kern
Message-ID:  <200902261014.n1QAEAv9024396@svn.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: ed
Date: Thu Feb 26 10:14:10 2009
New Revision: 189060
URL: http://svn.freebsd.org/changeset/base/189060

Log:
  Don't use PTY name as format string, even though it isn't insecure here.
  
  It's guaranteed that the `name' variable always contains a string of the
  form pty[l‐sL‐S][0‐9a‐v], but I'd rather keep the compiler happy (LLVM).

Modified:
  head/sys/kern/tty_pty.c

Modified: head/sys/kern/tty_pty.c
==============================================================================
--- head/sys/kern/tty_pty.c	Thu Feb 26 06:16:15 2009	(r189059)
+++ head/sys/kern/tty_pty.c	Thu Feb 26 10:14:10 2009	(r189060)
@@ -113,7 +113,7 @@ pty_clone(void *arg, struct ucred *cr, c
 
 	/* Create the controller device node. */
 	*dev = make_dev_credf(MAKEDEV_REF, &ptydev_cdevsw, 0,
-	    NULL, UID_ROOT, GID_WHEEL, 0666, name);
+	    NULL, UID_ROOT, GID_WHEEL, 0666, "%s", name);
 }
 
 static void



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