diff options
author | yoav-steinberg <yoav@monfort.co.il> | 2021-09-26 17:45:02 +0300 |
---|---|---|
committer | GitHub <noreply@github.com> | 2021-09-26 17:45:02 +0300 |
commit | 66002530466a45bce85e4930364f1b153c44840b (patch) | |
tree | eb5f2031208ed3ca927c2b08b107e05611c17ebe /src/blocked.c | |
parent | 0af7fe2cab04309e57733b428817f477d06ecbda (diff) | |
download | redis-66002530466a45bce85e4930364f1b153c44840b.tar.gz |
Client eviction ci issues (#9549)
Fixing CI test issues introduced in #8687
- valgrind warnings in readQueryFromClient when client was freed by processInputBuffer
- adding DEBUG pause-cron for tests not to be time dependent.
- skipping a test that depends on socket buffers / events not compatible with TLS
- making sure client got subscribed by not using deferring client
Diffstat (limited to 'src/blocked.c')
-rw-r--r-- | src/blocked.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/src/blocked.c b/src/blocked.c index 86aed2440..67fd3fdca 100644 --- a/src/blocked.c +++ b/src/blocked.c @@ -141,8 +141,10 @@ void processUnblockedClients(void) { if (processPendingCommandsAndResetClient(c) == C_OK) { /* Now process client if it has more data in it's buffer. */ if (c->querybuf && sdslen(c->querybuf) > 0) { - processInputBuffer(c); + if (processInputBuffer(c) == C_ERR) c = NULL; } + } else { + c = NULL; } } beforeNextClient(c); |