summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorOran Agra <oran@redislabs.com>2020-02-06 09:33:20 +0200
committerOran Agra <oran@redislabs.com>2020-02-06 09:33:20 +0200
commitd454d5a4f516991c0f1e21a285c71f38a113e297 (patch)
treee80f0d8e7f7835cf8ba6cbd72f379e8f0ed4a928
parent44ac202fbfbca4210d016c9f77df987b27c1ae4c (diff)
downloadredis-d454d5a4f516991c0f1e21a285c71f38a113e297.tar.gz
Fix client flags to be int64 in module.c
currently there's no bug since the flags these functions handle are always lower than 32bit, but still better fix the type to prevent future bugs.
-rw-r--r--src/module.c6
1 files changed, 3 insertions, 3 deletions
diff --git a/src/module.c b/src/module.c
index 914c50df3..a2bc869f9 100644
--- a/src/module.c
+++ b/src/module.c
@@ -714,9 +714,9 @@ void RM_KeyAtPos(RedisModuleCtx *ctx, int pos) {
* flags into the command flags used by the Redis core.
*
* It returns the set of flags, or -1 if unknown flags are found. */
-int commandFlagsFromString(char *s) {
+int64_t commandFlagsFromString(char *s) {
int count, j;
- int flags = 0;
+ int64_t flags = 0;
sds *tokens = sdssplitlen(s,strlen(s)," ",1,&count);
for (j = 0; j < count; j++) {
char *t = tokens[j];
@@ -798,7 +798,7 @@ int commandFlagsFromString(char *s) {
* to authenticate a client.
*/
int RM_CreateCommand(RedisModuleCtx *ctx, const char *name, RedisModuleCmdFunc cmdfunc, const char *strflags, int firstkey, int lastkey, int keystep) {
- int flags = strflags ? commandFlagsFromString((char*)strflags) : 0;
+ int64_t flags = strflags ? commandFlagsFromString((char*)strflags) : 0;
if (flags == -1) return REDISMODULE_ERR;
if ((flags & CMD_MODULE_NO_CLUSTER) && server.cluster_enabled)
return REDISMODULE_ERR;