From: Paul Jackson Date: Sun, 17 Apr 2005 01:15:30 GMT Subject: Re: fix mktemp (remove mktemp ;) Message-ID: <20050416181530.6ccd569b.pj@sgi.com> In-Reply-To: <20050417005757.GB15608@redhat.com> Dave wrote: > http://www.linuxsecurity.com/content/view/115462/151/ Nice - thanks. Pasky - would you be interested in a patch that used a more robust tmp file creation, along the lines of replacing t=${TMPDIR:-/usr/tmp}/gitdiff.$$ trap 'set +f; rm -fr $t.?; trap 0; exit 0' 0 1 2 3 15 with: tmp=${TMPDIR-/tmp} tmp=$tmp/gitdiff-do.$RANDOM.$RANDOM.$RANDOM.$$ (umask 077 && mkdir $tmp) || { echo "Could not create temporary directory! Exiting." 1>&2 exit 1 } t=$tmp/tmp trap 'rm -fr $tmp; trap 0; exit 0' 0 1 2 3 15 If interested, would you want it instead of my previous mktemp removal patch, or on top of it? -- I won't rest till it's the best ... Programmer, Linux Scalability Paul Jackson 1.650.933.1373, 1.925.600.0401