From 7229fa8d6d8acdedf0383de226b9b91a7d80197d Mon Sep 17 00:00:00 2001 From: antirez Date: Fri, 24 Nov 2017 13:29:54 +0100 Subject: [PATCH] Modules: fix memory leak in RM_IsModuleNameBusy(). --- src/module.c | 10 +++------- 1 file changed, 3 insertions(+), 7 deletions(-) diff --git a/src/module.c b/src/module.c index 7f9402ec..8a4c40f1 100644 --- a/src/module.c +++ b/src/module.c @@ -664,13 +664,9 @@ void RM_SetModuleAttribs(RedisModuleCtx *ctx, const char *name, int ver, int api * Otherwise zero is returned. */ int RM_IsModuleNameBusy(const char *name) { sds modulename = sdsnew(name); - - /* Check if the module name is busy. */ - if (dictFind(modules,modulename) != NULL) { - sdsfree(modulename); - return 1; - } - return 0; + dictEntry *de = dictFind(modules,modulename); + sdsfree(modulename); + return de != NULL; } /* Return the current UNIX time in milliseconds. */