[rrd-developers] rrdcached: path sanitizer

Tobias Oetiker tobi at oetiker.ch
Thu Dec 6 06:39:54 CET 2012


Today Kiss Gabor (Bitman) wrote:

> > >        realpath() expands all symbolic links and resolves references  to
> > >  /./,
>
> > This is going to cause a LOT of stat() calls: O(commands * average dir
> > depth)
> >
> > I see your point, but it's probably better to do trivial string
> > manipulations that don't hit the filesystem.
> > The whole point of rrdcached is to defer and collate I/O.
>
> You are absolutely right. Originally I wanted a simple character
> copy function but suddenly I found realpath() and I was too happy
> to think over what happens to symlinks. Sorry for the noise.

for the rrdtool future, make sure to check out and provide your
input on

https://github.com/oetiker/rrdtool-2.x

cheers
tobi
>
> Gabor
>
>

-- 
Tobi Oetiker, OETIKER+PARTNER AG, Aarweg 15 CH-4600 Olten, Switzerland
http://it.oetiker.ch tobi at oetiker.ch ++41 62 775 9902 / sb: -9900



More information about the rrd-developers mailing list