Fix AOF race that may duplicate commands

redis · Sep 16, 2011 · 5d46759 · 5d46759 · hoterran · Oct 29, 2011
1 parent 9e087a2
commit 5d46759
Showing 1 changed file with 5 additions and 0 deletions.
diff --git a/src/aof.c b/src/aof.c
@@ -657,6 +657,11 @@ void backgroundRewriteDoneHandler(int statloc) {
             if (server.appendfsync != APPENDFSYNC_NO) aof_fsync(fd);
             server.appendseldb = -1; /* Make sure it will issue SELECT */
             redisLog(REDIS_NOTICE,"The new append only file was selected for future appends.");
+
+            /* Clear regular AOF buffer since its contents was just written to
+             * the new AOF from the background rewrite buffer. */
+            sdsfree(server.aofbuf);
+            server.aofbuf = sdsempty();
         } else {
             /* If append only is disabled we just generate a dump in this
              * format. Why not? */