Static analysis complains that 'rc' was overwritten immediately after assigning to it, and therefore the assignment is useless. Remove it.
Cc: Dan Williams <[email protected]> Signed-off-by: Vishal Verma <[email protected]> --- test/dsm-fail.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) 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", -- 2.17.0 _______________________________________________ Linux-nvdimm mailing list [email protected] https://lists.01.org/mailman/listinfo/linux-nvdimm
