Message ID | 20180625053858.5927-1-vishal.l.verma@intel.com (mailing list archive) |
---|---|
State | New, archived |
Headers | show |
diff --git a/test/dsm-fail.c b/test/dsm-fail.c index 45a6c4f..6845f81 100644 --- a/test/dsm-fail.c +++ b/test/dsm-fail.c @@ -339,11 +339,9 @@ static int do_test(struct ndctl_ctx *ctx, struct ndctl_test *test) sprintf(path, "%s/fail_cmd", DIMM_PATH); sprintf(buf, "0\n"); rc = __sysfs_write_attr(&log_ctx, path, buf); - if (rc) { + if (rc) fprintf(stderr, "%s: failed to clear fail_cmd mask\n", ndctl_dimm_get_devname(victim)); - rc = -ENXIO; - } rc = ndctl_dimm_enable(victim); if (rc) { fprintf(stderr, "failed to enable victim: %s after clearing error\n",
Static analysis complains that 'rc' was overwritten immediately after assigning to it, and therefore the assignment is useless. Remove it. Cc: Dan Williams <dan.j.williams@intel.com> Signed-off-by: Vishal Verma <vishal.l.verma@intel.com> --- test/dsm-fail.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-)