[Sheepdog] [PATCH 2/4] cluster: add local cluster driver

[Chris Webb]

MORITA Kazutaka <morita.kazutaka at lab.ntt.co.jp> writes:

> At Sat, 29 Oct 2011 11:10:29 +0100,
> Chris Webb wrote:
> > 
> > MORITA Kazutaka <morita.kazutaka at lab.ntt.co.jp> writes:
> > 
> > > +const char *shmfile = "/tmp/sheepdog_shm";
> > [...]
> > > +	shmfd = open(shmfile, O_CREAT | O_RDWR, 0644);
> > 
> > Even though this is just a testing driver, this does make me a little bit
> > nervous. For instance, as a malicious user of your machine, I might have
> > done
> > 
> >   ln -s /etc/shadow /tmp/sheepdog_shm
> > 
> > before you started sheepdog.
> 
> How about specifying a shared file name in the command line argument?
> For example:
> 
>  $ sheep /store -c local:/share/file/name

Hi Kazutaka. That sounds like a good solution to me.

Best wishes,

Chris.