diff options
author | antirez <antirez@gmail.com> | 2019-03-15 17:16:06 +0100 |
---|---|---|
committer | antirez <antirez@gmail.com> | 2019-03-15 17:16:06 +0100 |
commit | e216ceaf0e099536fe3658a29dcb725d812364e0 (patch) | |
tree | 46aaf159df2c3da0a6f5b2c493a1f9ddea95cfe9 /src | |
parent | dca7358279bb6449f93e01f7d2806639b8e9ec4b (diff) | |
download | redis-e216ceaf0e099536fe3658a29dcb725d812364e0.tar.gz |
HyperLogLog: handle wrong offset in the base case.
Diffstat (limited to 'src')
-rw-r--r-- | src/hyperloglog.c | 8 |
1 files changed, 2 insertions, 6 deletions
diff --git a/src/hyperloglog.c b/src/hyperloglog.c index 526510b43..1e7ce3dce 100644 --- a/src/hyperloglog.c +++ b/src/hyperloglog.c @@ -614,10 +614,7 @@ int hllSparseToDense(robj *o) { } else { runlen = HLL_SPARSE_VAL_LEN(p); regval = HLL_SPARSE_VAL_VALUE(p); - if ((runlen + idx) > HLL_REGISTERS) { - sdsfree(dense); - return C_ERR; - } + if ((runlen + idx) > HLL_REGISTERS) break; /* Overflow. */ while(runlen--) { HLL_DENSE_SET_REGISTER(hdr->registers,idx,regval); idx++; @@ -1097,8 +1094,7 @@ int hllMerge(uint8_t *max, robj *hll) { } else { runlen = HLL_SPARSE_VAL_LEN(p); regval = HLL_SPARSE_VAL_VALUE(p); - if ((runlen + i) > HLL_REGISTERS) - return C_ERR; + if ((runlen + i) > HLL_REGISTERS) break; /* Overflow. */ while(runlen--) { if (regval > max[i]) max[i] = regval; i++; |