diff options
author | WuYunlong <xzsyeb@126.com> | 2020-08-11 10:18:21 +0800 |
---|---|---|
committer | GitHub <noreply@github.com> | 2020-08-10 19:18:21 -0700 |
commit | d6220f12a9fafeb07ad86fa91e88383ca2a524c8 (patch) | |
tree | 3e32c9a3520058951fe47d51380f4df4aa1ae090 /src/redismodule.h | |
parent | 3941a275f7994bc5563b442bb45264abee7511ba (diff) | |
download | redis-d6220f12a9fafeb07ad86fa91e88383ca2a524c8.tar.gz |
see #7250, fix signature of RedisModule_DeauthenticateAndCloseClient (#7645)
In redismodule.h, RedisModule_DeauthenticateAndCloseClient returns void
`void REDISMODULE_API_FUNC(RedisModule_DeauthenticateAndCloseClient)(RedisModuleCtx *ctx, uint64_t client_id);`
But in module.c, RM_DeauthenticateAndCloseClient returns int
`int RM_DeauthenticateAndCloseClient(RedisModuleCtx *ctx, uint64_t client_id)`
It it safe to change return value from `void` to `int` from the user's perspective.
Diffstat (limited to 'src/redismodule.h')
-rw-r--r-- | src/redismodule.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/redismodule.h b/src/redismodule.h index 2f2213677..5f828b9e3 100644 --- a/src/redismodule.h +++ b/src/redismodule.h @@ -680,7 +680,7 @@ void REDISMODULE_API_FUNC(RedisModule_FreeModuleUser)(RedisModuleUser *user); int REDISMODULE_API_FUNC(RedisModule_SetModuleUserACL)(RedisModuleUser *user, const char* acl); int REDISMODULE_API_FUNC(RedisModule_AuthenticateClientWithACLUser)(RedisModuleCtx *ctx, const char *name, size_t len, RedisModuleUserChangedFunc callback, void *privdata, uint64_t *client_id); int REDISMODULE_API_FUNC(RedisModule_AuthenticateClientWithUser)(RedisModuleCtx *ctx, RedisModuleUser *user, RedisModuleUserChangedFunc callback, void *privdata, uint64_t *client_id); -void REDISMODULE_API_FUNC(RedisModule_DeauthenticateAndCloseClient)(RedisModuleCtx *ctx, uint64_t client_id); +int REDISMODULE_API_FUNC(RedisModule_DeauthenticateAndCloseClient)(RedisModuleCtx *ctx, uint64_t client_id); #endif #define RedisModule_IsAOFClient(id) ((id) == CLIENT_ID_AOF) |