main - vdo: suffle code for better error path handling

[Zdenek Kabelac]

Gitweb:        https://sourceware.org/git/?p=lvm2.git;a=commitdiff;h=493acb9195cef185b38ae4...
Commit:        493acb9195cef185b38ae4e4ffb84b984e5cc08c
Parent:        e2e31d9acf1b96ab741c22dc0a2fefd672996d3a
Author:        Zdenek Kabelac 
AuthorDate:    Sat Jul 9 21:33:57 2022 +0200
Committer:     Zdenek Kabelac 
CommitterDate: Mon Jul 11 01:18:24 2022 +0200

vdo: suffle code for better error path handling

For failing dm_ no need to report 2nd. error,
but we missed to report error with 'updated==NULL'.
---
 tools/toollib.c | 28 ++++++++++++++--------------
 1 file changed, 14 insertions(+), 14 deletions(-)

diff --git a/tools/toollib.c b/tools/toollib.c
index 544791808..c29ba2ab4 100644
--- a/tools/toollib.c
+++ b/tools/toollib.c
@@ -1347,23 +1347,23 @@ int get_vdo_settings(struct cmd_context *cmd,
    		u |= VDO_CHANGE_ONLINE;
    }

-	if (updated) {
-		// validation of updated VDO option
-		if (!dm_vdo_validate_target_params(vtp, 0 /* vdo_size */)) {
-err:
-			if (is_lvchange)
-				log_error("Cannot change VDO setting "vdo_%s" in existing VDO pool.",
-					  option);
-			else
-				log_error("Invalid argument for VDO setting "vdo_%s".",
-					  option);
-			goto out;
-		}
+	// validation of updated VDO option
+	if (!dm_vdo_validate_target_params(vtp, 0 /* vdo_size */))
+		goto_out;

+	if (updated)
    	*updated = u;
-	}

-	r = 1;
+	r = 1; // success
+	goto out;
+err:
+	if (is_lvchange)
+		log_error("Cannot change VDO setting "vdo_%s" in existing VDO pool.",
+			  option);
+	else
+		log_error("Invalid argument for VDO setting "vdo_%s".",
+			  option);
+
 out:
    if (result)
    	dm_config_destroy(result);