mirror of
https://github.com/fluencelabs/redis
synced 2025-03-19 09:00:51 +00:00
Clarify comment in change fixing #4323.
This commit is contained in:
parent
fa2c0582cf
commit
474adba9fa
@ -75,8 +75,12 @@ slowlogEntry *slowlogCreateEntry(client *c, robj **argv, int argc, long long dur
|
|||||||
} else if (argv[j]->refcount == OBJ_SHARED_REFCOUNT) {
|
} else if (argv[j]->refcount == OBJ_SHARED_REFCOUNT) {
|
||||||
se->argv[j] = argv[j];
|
se->argv[j] = argv[j];
|
||||||
} else {
|
} else {
|
||||||
/* Duplicate a string object,
|
/* Here we need to dupliacate the string objects composing the
|
||||||
* avoid memory leak for lazyfree. */
|
* argument vector of the command, because those may otherwise
|
||||||
|
* end shared with string objects stored into keys. Having
|
||||||
|
* shared objects between any part of Redis, and the data
|
||||||
|
* structure holding the data, is a problem: FLUSHALL ASYNC
|
||||||
|
* may release the shared string object and create a race. */
|
||||||
se->argv[j] = dupStringObject(argv[j]);
|
se->argv[j] = dupStringObject(argv[j]);
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
Loading…
x
Reference in New Issue
Block a user