Explicitly check C_ERR condition to improve readability in clusterSav…

…eConfig (#1505) It's not obvious to see it at first, modify it to use C_ERR. Signed-off-by: Binbin <[email protected]>
valkey-io · Jan 4, 2025 · 33b8241 · 33b8241
1 parent b3b4bdc
commit 33b8241
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/src/cluster_legacy.c b/src/cluster_legacy.c
@@ -868,7 +868,7 @@ int clusterSaveConfig(int do_fsync) {
 
 cleanup:
     if (fd != -1) close(fd);
-    if (retval) unlink(tmpfilename);
+    if (retval == C_ERR) unlink(tmpfilename);
     sdsfree(tmpfilename);
     sdsfree(ci);
     return retval;