On 21/09/14 18:31, Michał Górny wrote: > Dnia 2014-09-21, o godz. 11:52:25 > Diego Biurrun <[email protected]> napisał(a): > >> This reduces TMPDIR pollution and possibly avoids race conditions with >> temporary files that are not atomically created. >> >> Based on a patch from Michał Górny <[email protected]>. >> --- >> >> Rewritten to be less intrusive. >> >> configure | 11 +++++++---- >> 1 file changed, 7 insertions(+), 4 deletions(-) >> >> diff --git a/configure b/configure >> index 59dd453..ea3244c 100755 >> --- a/configure >> +++ b/configure >> @@ -2593,15 +2593,18 @@ if ! check_cmd mktemp -u XXXXXX; then >> } >> fi >> >> +AVTMPDIR=$(mktemp -u "${TMPDIR}/avconf.XXXXXXXX" 2> /dev/null) >> + >> +mkdir $AVTMPDIR || die "Unable to create temporary directory in $TMPDIR." >> + > > Your patch no longer avoids race conditions since once again you are > using 'mktemp -u'. In fact, I don't understand at all why you are > creating the directory manually when 'mktemp -d' will do that for you.
Thanks a lot for spotting! lu _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
