public inbox for cluster-cvs@sourceware.org
help / color / mirror / Atom feed
* cluster: RHEL4 - rgmanager: fix ambiguous output when resource fails to stop
@ 2009-03-12 21:40 Lon Hohberger
  0 siblings, 0 replies; only message in thread
From: Lon Hohberger @ 2009-03-12 21:40 UTC (permalink / raw)
  To: cluster-cvs-relay

Gitweb:        http://git.fedorahosted.org/git/cluster.git?p=cluster.git;a=commitdiff;h=1e89c485ec2de1f91727569ba3230b36b26034d3
Commit:        1e89c485ec2de1f91727569ba3230b36b26034d3
Parent:        46a8552a07c7e72122c9495c74eb97276a31b7bc
Author:        Lon Hohberger <lhh@redhat.com>
AuthorDate:    Mon Mar 2 15:36:00 2009 -0500
Committer:     Lon Hohberger <lhh@redhat.com>
CommitterDate: Thu Mar 12 17:40:09 2009 -0400

rgmanager: fix ambiguous output when resource fails to stop

https://bugzilla.redhat.com/show_bug.cgi?id=480835

Signed-off-by: Lon Hohberger <lhh@redhat.com>
---
 rgmanager/src/daemons/rg_state.c |    5 ++++-
 1 files changed, 4 insertions(+), 1 deletions(-)

diff --git a/rgmanager/src/daemons/rg_state.c b/rgmanager/src/daemons/rg_state.c
index df0ac1b..c717ecc 100644
--- a/rgmanager/src/daemons/rg_state.c
+++ b/rgmanager/src/daemons/rg_state.c
@@ -974,7 +974,10 @@ _svc_stop(char *svcName, int req, int recover, uint32_t newstate)
 	}
 	rg_unlock(svcName, lockp);
 
-	ret = group_op(svcName, RG_STOP);
+	if (group_op(svcName, RG_STOP) != 0)
+		ret = RG_EFAIL;
+	else
+		ret = 0;
 
 	if (old_state == RG_STATE_FAILED && newstate == RG_STATE_DISABLED) {
 		if (ret)


^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2009-03-12 21:40 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2009-03-12 21:40 cluster: RHEL4 - rgmanager: fix ambiguous output when resource fails to stop Lon Hohberger

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).