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