diff options
author | Fabio M. Di Nitto <fdinitto@redhat.com> | 2010-08-12 09:14:59 +0000 |
---|---|---|
committer | Fabio M. Di Nitto <fdinitto@redhat.com> | 2010-08-12 09:14:59 +0000 |
commit | 2565ffad25808c25cbf5509e8c5ffc2a1c05671f (patch) | |
tree | 473db1922fdd3d4d297670eb075c01cf97ea54df /udev | |
parent | 9ac9e06b5c4d5684248e0c117428f5ccafb40377 (diff) | |
download | lvm2-2565ffad25808c25cbf5509e8c5ffc2a1c05671f.tar.gz lvm2-2565ffad25808c25cbf5509e8c5ffc2a1c05671f.tar.xz lvm2-2565ffad25808c25cbf5509e8c5ffc2a1c05671f.zip |
Fix clvmd init script return code when executed as non-root user.
clvmd daemon itself does the right thing when invoked as non-root, by
returning 4.
The patch removes the use daemon function from
/etc/rc.d/init.d/functions that´s unnecessary and has th bad habit to
mask the return codes from the real daemon.
Add a simple and generic check to see if clvmd is executed by root or not.
Our stop/reload/restart paths in the init script are complex and not all
the tools involved in the process are guaranteed to return 4 if executed
by non-root against a process that´s running as root (for example kill
-TERM will return -1 and parsing the output to catch the error is
suboptimal at best).
https://bugzilla.redhat.com/show_bug.cgi?id=553381
Diffstat (limited to 'udev')
0 files changed, 0 insertions, 0 deletions