Re: rrdcached: path sanitizer

"Kiss Gabor (Bitman)" <[email protected]> Thu, 6 Dec 2012 06:31:47 +0100 (CET)
Newsgroups gmane.comp.db.rrdtool.devel
Message-ID <[email protected]>
> >        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.

Gabor