caleb-brown/redict
1
0
Fork 0
mirror of https://codeberg.org/redict/redict.git synced 2025-01-22 16:18:28 -05:00
Code Issues Actions Packages Projects Releases Wiki Activity

Merge pull request #2224 from advance512/spopWithCount

Browse Source 
Fix: case when SPOP with count>MAXINT, setTypeRandomElements() will get ...
This commit is contained in:
Salvatore Sanfilippo 2014-12-18 15:11:44 +01:00
commit efbf5a125e
2 changed files with 7 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/redis.h
View File

@ -1269,7 +1269,7 @@ void setTypeReleaseIterator(setTypeIterator *si);
int setTypeNext(setTypeIterator *si, robj **objele, int64_t *llele);
robj *setTypeNextObject(setTypeIterator *si);
int setTypeRandomElement(robj *setobj, robj **objele, int64_t *llele);
int setTypeRandomElements(robj *set, int count, robj *aux_set);
unsigned long setTypeRandomElements(robj *set, unsigned long count, robj *aux_set);
unsigned long setTypeSize(robj *subject);
void setTypeConvert(robj *subject, int enc);

12
src/t_set.c
View File

@ -225,11 +225,11 @@ int setTypeRandomElement(robj *setobj, robj **objele, int64_t *llele) {
* at producing N elements, and the elements are guaranteed to be non
* repeating.
*/
int setTypeRandomElements(robj *set, int count, robj *aux_set) {
int set_size;
int elements_to_return = count;
int elements_copied = 0;
int current_element = 0;
unsigned long setTypeRandomElements(robj *set, unsigned long count, robj *aux_set) {
unsigned long set_size;
unsigned long elements_to_return = count;
unsigned long elements_copied = 0;
unsigned long current_element = 0;
/* Like all setType* functions, we assume good behavior on part of the caller,
* so no extra parameter checks are made. */
@ -480,7 +480,7 @@ void scardCommand(redisClient *c) {
void spopWithCountCommand(redisClient *c) {
long l;
unsigned long count, size;
int elements_returned;
unsigned long elements_returned;
robj *set, *aux, *aux_set;
int64_t llele;