From fe86f890b07a124afdc0417bb0089677a892d454 Mon Sep 17 00:00:00 2001
From: antirez <antirez@gmail.com>
Date: Mon, 13 Jan 2014 11:55:57 +0100
Subject: [PATCH] SENTINEL SET: error on bad option name + flush config on
 error.

---
 src/sentinel.c | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/src/sentinel.c b/src/sentinel.c
index f2fe1540c..ca538908d 100644
--- a/src/sentinel.c
+++ b/src/sentinel.c
@@ -2644,6 +2644,7 @@ void sentinelSetCommand(redisClient *c) {
             if (strlen(value) && access(value,X_OK) == -1) {
                 addReplyError(c,
                     "Notification script seems non existing or non executable");
+                if (changes) sentinelFlushConfig();
                 return;
             }
             sdsfree(ri->notification_script);
@@ -2655,6 +2656,7 @@ void sentinelSetCommand(redisClient *c) {
                 addReplyError(c,
                     "Client reconfiguration script seems non existing or "
                     "non executable");
+                if (changes) sentinelFlushConfig();
                 return;
             }
             sdsfree(ri->client_reconfig_script);
@@ -2665,6 +2667,11 @@ void sentinelSetCommand(redisClient *c) {
             sdsfree(ri->auth_pass);
             ri->auth_pass = strlen(value) ? sdsnew(value) : NULL;
             changes++;
+        } else {
+            addReplyErrorFormat(c,"Unknown option '%s' for SENTINEL SET",
+                option);
+            if (changes) sentinelFlushConfig();
+            return;
         }
     }
 
-- 
GitLab