Cristian Rodríguez wrote on 14/01/15 05:51: > Otherwise, if the directory contains other directories we fail > at fopen in load_sysv() with EISDIR.
> - if (hidden_file(de->d_name)) > - continue; Why are hidden files now processed? I'd still suggest you want to skip files ending in ~ at very least, but I don't think any files beginning with a . should be processed either. > + dirent_ensure_type(d, de); Should the return value be checked here? It's quite unlikely I know, so maybe not. > + > + if (!dirent_is_file(de)) > + continue; If I read the dirent_ensure_type and dirent_is_file code properly, this would mean that symlinks to valid sysvinit scripts are now skipped. Is this intended? I'd say someone will complain about that with some kind of wierd setup (e.g. a symlink to some custom init script in /opt or similar). Col -- Colin Guthrie gmane(at)colin.guthr.ie http://colin.guthr.ie/ Day Job: Tribalogic Limited http://www.tribalogic.net/ Open Source: Mageia Contributor http://www.mageia.org/ PulseAudio Hacker http://www.pulseaudio.org/ Trac Hacker http://trac.edgewall.org/ _______________________________________________ systemd-devel mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/systemd-devel
