[lvm-devel] master - gcc: all paths will set ret

Zdenek Kabelac zkabelac at sourceware.org
Mon Oct 21 13:33:31 UTC 2019


Gitweb:        https://sourceware.org/git/?p=lvm2.git;a=commitdiff;h=644186e9204bfca21a6b3ca9dcc8256b93a74c15
Commit:        644186e9204bfca21a6b3ca9dcc8256b93a74c15
Parent:        f61d828c86d2b6466224290b9a1ffdaa16419f3d
Author:        Zdenek Kabelac <zkabelac at redhat.com>
AuthorDate:    Mon Oct 21 12:25:04 2019 +0200
Committer:     Zdenek Kabelac <zkabelac at redhat.com>
CommitterDate: Mon Oct 21 15:32:35 2019 +0200

gcc: all paths will set ret

Set success on common path.
Fixes random failure on writecache uncaching path.
---
 tools/lvconvert.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/tools/lvconvert.c b/tools/lvconvert.c
index e033e71..c312956 100644
--- a/tools/lvconvert.c
+++ b/tools/lvconvert.c
@@ -4757,7 +4757,7 @@ static int _lvconvert_split_cache_single(struct cmd_context *cmd,
 			if (lvremove_single(cmd, lv_fast, NULL) != ECMD_PROCESSED)
 				return ECMD_FAILED;
 		}
-
+		ret = 1;
 	} else if (lv_is_cache(lv_main) && lv_is_cache_vol(lv_fast)) {
 		if (cmd->command->command_enum == lvconvert_split_and_remove_cache_CMD) {
 			ret = _lvconvert_split_and_remove_cachevol(cmd, lv_main, lv_fast);




More information about the lvm-devel mailing list