From: antirez <antirez@gmail.com>
Date: Mon, 23 Mar 2009 20:33:15 +0000 (+0100)
Subject: Now MONITOR/SYNC cannot be issued multiple times
X-Git-Url: https://git.saurik.com/redis.git/commitdiff_plain/cf3f0c012df3abeee9bc2ec511820c1a859ecfc7

Now MONITOR/SYNC cannot be issued multiple times
---

diff --git a/TODO b/TODO
index 7d91a103..9bf8644f 100644
--- a/TODO
+++ b/TODO
@@ -34,3 +34,5 @@ ROLLBACK command:
     COMMIT
 
     but this sucks since there is no way to check the error message.
+
+- Prevent the client to issue SYNC or MONITOR multiple times
diff --git a/redis.c b/redis.c
index c7b10dfd..0fc66795 100644
--- a/redis.c
+++ b/redis.c
@@ -2919,6 +2919,9 @@ static void syncCommand(redisClient *c) {
     time_t start = time(NULL);
     char sizebuf[32];
 
+    /* ignore SYNC if aleady slave or in monitor mode */
+    if (c->flags & REDIS_SLAVE) return;
+
     redisLog(REDIS_NOTICE,"Slave ask for syncronization");
     if (flushClientOutput(c) == REDIS_ERR || saveDb(server.dbfilename) != REDIS_OK)
         goto closeconn;
@@ -3029,6 +3032,9 @@ static int syncWithMaster(void) {
 }
 
 static void monitorCommand(redisClient *c) {
+    /* ignore MONITOR if aleady slave or in monitor mode */
+    if (c->flags & REDIS_SLAVE) return;
+
     c->flags |= (REDIS_SLAVE|REDIS_MONITOR);
     c->slaveseldb = 0;
     if (!listAddNodeTail(server.monitors,c)) oom("listAddNodeTail");