mirror of
https://github.com/fluencelabs/redis
synced 2025-03-17 16:10:50 +00:00
fixed bad transfer of ownership in HashSet causing a potential crash
This commit is contained in:
parent
9aff564045
commit
cfaef8d5d4
11
src/module.c
11
src/module.c
@ -1795,10 +1795,13 @@ int RM_HashSet(RedisModuleKey *key, int flags, ...) {
|
||||
if (flags & REDISMODULE_HASH_CFIELDS)
|
||||
low_flags |= HASH_SET_TAKE_FIELD;
|
||||
updated += hashTypeSet(key->value, field->ptr, value->ptr, low_flags);
|
||||
field->ptr = NULL; /* Ownership is now of hashTypeSet() */
|
||||
|
||||
/* Cleanup */
|
||||
if (flags & REDISMODULE_HASH_CFIELDS) decrRefCount(field);
|
||||
|
||||
/* If CFIELDS is active, ownership is now of hashTypeSet() */
|
||||
if (flags & REDISMODULE_HASH_CFIELDS) {
|
||||
field->ptr = NULL;
|
||||
/* Cleanup */
|
||||
decrRefCount(field);
|
||||
}
|
||||
}
|
||||
va_end(ap);
|
||||
moduleDelKeyIfEmpty(key);
|
||||
|
Loading…
x
Reference in New Issue
Block a user