Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 7 Nov 2000 15:18:00 -0500 (EST)
From:      Garrett Wollman <wollman@khavrinen.lcs.mit.edu>
To:        Garance A Drosihn <drosih@rpi.edu>
Cc:        audit@FreeBSD.org, freebsd-print@bostonradio.org
Subject:   Fixing mktemp() call in lpd/printjob.c (logging)
Message-ID:  <200011072018.PAA86497@khavrinen.lcs.mit.edu>
In-Reply-To: <p04330101b62cfcffd73e@[128.113.24.47]>
References:  <20001024140510.G93799@lucifer.bart.nl> <20001024222716.B2020@lucifer.bart.nl> <p0433010cb62cc872490a@[128.113.24.47]> <p04330101b62cfcffd73e@[128.113.24.47]>

next in thread | previous in thread | raw e-mail | index | archive | help
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

<<On Mon, 6 Nov 2000 19:44:44 -0500, Garance A Drosihn <drosih@rpi.edu> said:

> Okay, I believe I have an update which replaces the call to
> mktemp with a call to mkstemp, without introducing any problems
> to printjob.

This seems to be one of those cases where the ``might be unsafe''
warning does us a disservice.  Since this file is created in the spool
directory, which is only writable by the spooler subsystem, mktemp()
is safe here.

That said, mkstemp() is a clearly superior interface.  I would,
however, prefer to see consistent style for variable names.

- -GAWollman

- --
Garrett A. Wollman   | O Siem / We are all family / O Siem / We're all the same
wollman@lcs.mit.edu  | O Siem / The fires of freedom 
Opinions not those of| Dance in the burning flame
MIT, LCS, CRS, or NSA|                     - Susan Aglukark and Chad Irschick
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.0.4 (FreeBSD)
Comment: For info see http://www.gnupg.org

iD8DBQE6CGNqI+eG6b7tlG4RAlakAKCMoUU175iMyOhrgdp6NeTMgAH9fwCcCLSB
9M2bHnGmuOAHsJXnQCH4kUQ=
=UG4d
-----END PGP SIGNATURE-----


To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe freebsd-audit" in the body of the message




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