[Varnish] #316: varnishtest should mkdtemp or similar for its varnishd testing directory

Varnish varnish-bugs at projects.linpro.no
Mon Sep 8 21:27:20 CEST 2008


#316: varnishtest should mkdtemp or similar for its varnishd testing directory
--------------------+-------------------------------------------------------
 Reporter:  ingvar  |       Owner:  des  
     Type:  defect  |      Status:  new  
 Priority:  normal  |   Milestone:       
Component:  build   |     Version:  trunk
 Severity:  normal  |    Keywords:       
--------------------+-------------------------------------------------------
 If one builds varnish first as root, breaks it while running 'make check',
 and then build and run make check as a non-privileged user, varnishtest
 dies on something like this:

 {{{
 Error: (-sfile) "." mkstemp(./varnish.UQTZnP) failed (Permission denied)
 }}}

 This is because it uses the same temporary catalog for testing, that is,
 in my builds,
 {{{
 /tmp/__v1
 }}}
 which is owned by root after the first run.

 The correct fix would be to use mkdtemp or similar in addition to mkstemp

 Ingvar

-- 
Ticket URL: <http://varnish.projects.linpro.no/ticket/316>
Varnish <http://varnish.projects.linpro.no/>
The Varnish HTTP Accelerator


More information about the varnish-bugs mailing list