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

Gitweb:        http://git.fedorahosted.org/git/cluster.git?p=cluster.git;a=commitdiff;h=e66f6d61deb7f97ad6994561b1ae918bd4aca8bc
Commit:        e66f6d61deb7f97ad6994561b1ae918bd4aca8bc
Parent:        8603c8797a5d989b653770b44555619c5d454cac
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:41:13 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:41 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:41 cluster: RHEL48 - 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).