[systemd-devel] [PATCH] journalctl: support /usr/bin/nginx, etc

[Lennart Poettering]

On Wed, 30.05.12 14:06, shawn (shawnlandden at gmail.com) wrote:

> 
> Regarding the commited version:
> 
> <snip>
> +                        if (S_ISREG(st.st_mode) && (0111 & st.st_mode))
> {
> </snip>
> 
> isn't it conceivable that some programs which would log would only be
> executable by the owner (generally root)? because they might be suid
> root? or then so we just revert to requiring the full _EXE= syntax?

(0111 & st.st_mode) returns non-zero when the x bit is set for either
the owner, the group or others. Hence the right thing should happen, no?

Lennart

-- 
Lennart Poettering - Red Hat, Inc.