diff options
author | antirez <antirez@gmail.com> | 2019-10-23 10:52:25 +0200 |
---|---|---|
committer | antirez <antirez@gmail.com> | 2019-10-23 10:52:29 +0200 |
commit | 856618570f65f6a4003a7791c89474cd884e7547 (patch) | |
tree | a396a90fae80b8e69a9251f828535ec73d128724 | |
parent | 3773f22bac712aaaf7e8691e5676dea495162d31 (diff) | |
download | redis-856618570f65f6a4003a7791c89474cd884e7547.tar.gz |
Modules hooks: don't call the hook for fake clients.
-rw-r--r-- | src/networking.c | 10 |
1 files changed, 7 insertions, 3 deletions
diff --git a/src/networking.c b/src/networking.c index dec49642f..e7cc561fa 100644 --- a/src/networking.c +++ b/src/networking.c @@ -1043,9 +1043,13 @@ void freeClient(client *c) { freeClientAsync(c); return; } - moduleFireServerEvent(REDISMODULE_EVENT_CLIENT_CHANGE, - REDISMODULE_SUBEVENT_CLIENT_CHANGE_DISCONNECTED, - c); + + /* For connected clients, call the disconnection event of modules hooks. */ + if (c->conn) { + moduleFireServerEvent(REDISMODULE_EVENT_CLIENT_CHANGE, + REDISMODULE_SUBEVENT_CLIENT_CHANGE_DISCONNECTED, + c); + } /* If it is our master that's beging disconnected we should make sure * to cache the state to try a partial resynchronization later. |