diff mbox

[05/11] ndctl: fix uninitialized variable in configure_namespace

Message ID 20151015210610.22046.76914.stgit@dwillia2-desk3.jf.intel.com (mailing list archive)
State Accepted
Commit 33090b7afb21
Headers show

Commit Message

Dan Williams Oct. 15, 2015, 9:06 p.m. UTC
In the pmem case lbasize is zero and the result of
ndctl_namespace_set_sector_size() is not relevant.

Signed-off-by: Dan Williams <dan.j.williams@intel.com>
---
 lib/test-libndctl.c |    9 +++++----
 1 file changed, 5 insertions(+), 4 deletions(-)
diff mbox

Patch

diff --git a/lib/test-libndctl.c b/lib/test-libndctl.c
index e2c72e3c3c25..93ceb358a301 100644
--- a/lib/test-libndctl.c
+++ b/lib/test-libndctl.c
@@ -651,11 +651,12 @@  static int configure_namespace(struct ndctl_region *region,
 			fprintf(stderr, "%s: set_size failed: %d\n", devname, rc);
 	}
 
-	if (lbasize)
+	if (lbasize) {
 		rc = ndctl_namespace_set_sector_size(ndns, lbasize);
-	if (rc)
-		fprintf(stderr, "%s: set_sector_size (%lu) failed: %d\n",
-			devname, lbasize, rc);
+		if (rc)
+			fprintf(stderr, "%s: set_sector_size (%lu) failed: %d\n",
+					devname, lbasize, rc);
+	}
 
 	rc = ndctl_namespace_is_configured(ndns);
 	if (rc < 1)