summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorSalvatore Sanfilippo <antirez@gmail.com>2018-04-13 19:51:46 +0200
committerGitHub <noreply@github.com>2018-04-13 19:51:46 +0200
commit7980d87c3c72740f4609fdcaae088221f8f8eb59 (patch)
tree2c185b21662acf2a47b8a639dcbe44ead0581095
parente07af6a2b782de809c1908a236332e48af5867b3 (diff)
parent775adf544c6ba27c099e16eb657607c9293fafa8 (diff)
downloadredis-7980d87c3c72740f4609fdcaae088221f8f8eb59.tar.gz
Merge pull request #4842 from soloestoy/fix-timer-api-wrong-insert
Modules Timer API: fix wrong raxInsert() usage
-rw-r--r--src/module.c5
1 files changed, 2 insertions, 3 deletions
diff --git a/src/module.c b/src/module.c
index 8c7544608..cb03ad2cd 100644
--- a/src/module.c
+++ b/src/module.c
@@ -4206,9 +4206,8 @@ RedisModuleTimerID RM_CreateTimer(RedisModuleCtx *ctx, mstime_t period, RedisMod
while(1) {
key = htonu64(expiretime);
- int retval = raxInsert(Timers,(unsigned char*)&key,sizeof(key),timer,NULL);
- if (retval) {
- expiretime = key;
+ if (raxFind(Timers, (unsigned char*)&key,sizeof(key)) == raxNotFound) {
+ raxInsert(Timers,(unsigned char*)&key,sizeof(key),timer,NULL);
break;
} else {
expiretime++;