On Thursday 20 September 2012 16:11:03 Tito wrote: > On Thursday 20 September 2012 15:58:07 you wrote: > > Tito <[email protected]> wrote: > > >This seems to work for me as expected eventually needs more testing. > > > > It doesn't work correctly in this case: > > > > $ mktemp -t -p /var/tmp -u temp.XXXXXX > > /var/tmp/temp.Jj1aWI > > $ export TMPDIR=/tmp > > $ mktemp -t -p /var/tmp -u temp.XXXXXX > > /tmp/temp.h8I9ly > > > > Ron > > > > Hi, > can you test commenting out this line? > > chp = tempnam(opts & (OPT_t|OPT_p) ? path : "./", chp); > if (!chp) > return EXIT_FAILURE; > ==> //if (!(opts & (OPT_t|OPT_p))) > chp = bb_get_last_path_component_strip(chp); > goto ret; > > Will repost the patch if this fixes your problem. > Ciao, > Tito
Forget about this. Seems to work like debian's mktemp with my patch applied, maybe needs more testing...... debian:~$ cd Desktop/busybox debian:~/Desktop/busybox$ mktemp -t -p /var/tmp -u temp.XXXXXX /var/tmp/temp.D0pKak debian:~/Desktop/busybox$ export TMPDIR=/tmp debian:~/Desktop/busybox$ mktemp -t -p /var/tmp -u temp.XXXXXX /tmp/temp.irsv2d debian:~/Desktop/busybox$ debian:~/Desktop/busybox$ ./busybox mktemp -t -p /var/tmp -u temp.XXXXXX mktemp: using fallback suid method /var/tmp/temp.OrU9HP debian:~/Desktop/busybox$ export TMPDIR=/tmp debian:~/Desktop/busybox$ ./busybox mktemp -t -p /var/tmp -u temp.XXXXXX mktemp: using fallback suid method /tmp/temp.kUb9NZ Ciao, Tito _______________________________________________ busybox mailing list [email protected] http://lists.busybox.net/mailman/listinfo/busybox
