All of lore.kernel.org
 help / color / mirror / Atom feed
* master - vgchange: fix error code in error path
@ 2018-06-14 20:02 Zdenek Kabelac
  0 siblings, 0 replies; only message in thread
From: Zdenek Kabelac @ 2018-06-14 20:02 UTC (permalink / raw)
  To: lvm-devel

Gitweb:        https://sourceware.org/git/?p=lvm2.git;a=commitdiff;h=70b159d145dfa960a53f6f61ce735b90ec25b1e0
Commit:        70b159d145dfa960a53f6f61ce735b90ec25b1e0
Parent:        3eff3aa4f8faa5a84de2ba696b15a976fad31852
Author:        Zdenek Kabelac <zkabelac@redhat.com>
AuthorDate:    Wed Jun 13 15:57:51 2018 +0200
Committer:     Zdenek Kabelac <zkabelac@redhat.com>
CommitterDate: Thu Jun 14 22:02:01 2018 +0200

vgchange: fix error code in error path

This rather hard to hit error path used wrong return value to signal
real error.
---
 tools/vgchange.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/tools/vgchange.c b/tools/vgchange.c
index 891f22a..b608d87 100644
--- a/tools/vgchange.c
+++ b/tools/vgchange.c
@@ -187,7 +187,7 @@ int vgchange_activate(struct cmd_context *cmd, struct volume_group *vg,
 	    strcmp(vg->system_id, cmd->system_id) &&
 	    do_activate) {
 		log_error("Cannot activate LVs in a foreign VG.");
-		return ECMD_FAILED;
+		return 0;
 	}
 
 	/*



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

only message in thread, other threads:[~2018-06-14 20:02 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-06-14 20:02 master - vgchange: fix error code in error path Zdenek Kabelac

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.