Unverified Commit 65c24bd3 authored by guybe7's avatar guybe7 Committed by GitHub
Browse files

Modules: Invalidate saved_oparray after use (#7688)

We wanna avoid a chance of someone using the pointer in it after it'll be freed / realloced.
parent cbaf3c5b
...@@ -609,6 +609,8 @@ void moduleHandlePropagationAfterCommandCallback(RedisModuleCtx *ctx) { ...@@ -609,6 +609,8 @@ void moduleHandlePropagationAfterCommandCallback(RedisModuleCtx *ctx) {
redisOpArrayFree(&server.also_propagate); redisOpArrayFree(&server.also_propagate);
/* Restore the previous oparray in case of nexted use of the API. */ /* Restore the previous oparray in case of nexted use of the API. */
server.also_propagate = ctx->saved_oparray; server.also_propagate = ctx->saved_oparray;
/* We're done with saved_oparray, let's invalidate it. */
redisOpArrayInit(&ctx->saved_oparray);
} }
} }
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment