gvsafronov/futriix
1
0
Fork 0
Code Issues Pull Requests Actions 1 Packages Projects Releases Wiki Activity

Merge pull request #7370 from oranagra/no_queue_in_aborted_multi

Browse Source 
Don't queue commands in an already aborted MULTI state
This commit is contained in:
Salvatore Sanfilippo 2020-06-08 11:08:08 +02:00 committed by GitHub
commit 254f06c131
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 7 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
src/multi.c
View File

@ -58,6 +58,13 @@ void queueMultiCommand(client *c) {
multiCmd *mc; multiCmd *mc;
int j; 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, c->mstate.commands = zrealloc(c->mstate.commands,
sizeof(multiCmd)*(c->mstate.count+1)); sizeof(multiCmd)*(c->mstate.count+1));
mc = c->mstate.commands+c->mstate.count; mc = c->mstate.commands+c->mstate.count;