currently, all module commands' proc point to the same function (RedisModuleCommandDispatcher) yet the actual command pointer is stored in getkeys_proc (which points to a RedisModuleCommand, which is a structure, not a function)

this whole scheme is a bit off and causes a great deal of confusion

Comment From: oranagra

i think it's ok that the proc is RedisModuleCommandDispatcher. but i think the module pointer should be stored in a proper field named module. i'm curious to know how many changes this fix will introduce.