Don't queue commands in an already aborted MULTI state

pmem · Jun 9, 2020 · e2046b3 · e2046b3
1 parent b35fdf1
commit e2046b3
Showing 1 changed file with 7 additions and 0 deletions.
diff --git a/src/multi.c b/src/multi.c
@@ -58,6 +58,13 @@ void queueMultiCommand(client *c) {
     multiCmd *mc;
     int j;
 
+    /* No sense to waste memory if the transaction is already aborted.
+     * this is useful in case client sends these in a pipeline, or doesn't
+     * bother to read previous responses and didn't notice the multi was already
+     * aborted. */
+    if (c->flags & CLIENT_DIRTY_EXEC)
+        return;
+
     c->mstate.commands = zrealloc(c->mstate.commands,
             sizeof(multiCmd)*(c->mstate.count+1));
     mc = c->mstate.commands+c->mstate.count;