[tor-dev] [Patch] src/test.c

[Steven J. Murdoch]

On Thu, Apr 07, 2011 at 05:10:05PM -0400, Nick Mathewson wrote:
> On Thu, Apr 7, 2011 at 3:48 PM, Gisle Vanem <gvanem at broadpark.no> wrote:
> > -               "c:\\windows\\temp\\tor_test_%d", (int)getpid());
> > +               "%s\\tor_test_%d", getenv("TEMP"), (int)getpid());
> 
> What guarantees that TEMP will always be defined?

A Windows (or DOS) system with %TEMP% undefined can reasonably be
expected to not work properly. On the other hand, a Windows system
which doesn't have a C: drive should work correctly. We could be a bit
defensive and if %TEMP% isn't defined, fall back on C:\WINDOWS\TEMP?

Steven.

-- 
http://www.cl.cam.ac.uk/users/sjm217/