diff options
author | Luca Berra <bluca@comedia.it> | 2010-02-27 16:07:51 +0100 |
---|---|---|
committer | NeilBrown <neilb@suse.de> | 2010-03-03 01:35:26 +0100 |
commit | cf556303573cd8cb9140c819b1ba1102ebc9cac7 (patch) | |
tree | 43d19c1aa1032df7a101244c2ac793a780ad18aa | |
parent | fix mdmon option parsing (diff) | |
download | mdadm-cf556303573cd8cb9140c819b1ba1102ebc9cac7.tar.xz mdadm-cf556303573cd8cb9140c819b1ba1102ebc9cac7.zip |
fix mdmon takeover
- when we waited for the old mdmon to exit, we didn't look
for the socket in the right place
- when we failed to find a pid file, we returned the wrong
value (code expected <0, but got ==0).
Signed-off-by: Luca Berra <bluca@comedia.it>
Signed-off-by: NeilBrown <neilb@suse.de>
-rw-r--r-- | msg.c | 2 | ||||
-rw-r--r-- | util.c | 4 |
2 files changed, 3 insertions, 3 deletions
@@ -147,7 +147,7 @@ int connect_monitor(char *devname) int pos; char *c; - pos = sprintf(path, "%s/", VAR_RUN); + pos = sprintf(path, "%s/", pid_dir); if (is_subarray(devname)) { devname++; c = strchr(devname, '/'); @@ -1478,11 +1478,11 @@ int mdmon_pid(int devnum) fd = open(path, O_RDONLY | O_NOATIME, 0); if (fd < 0) - return 0; + return -1; n = read(fd, pid, 9); close(fd); if (n <= 0) - return 0; + return -1; return atoi(pid); } |