gvsafronov/futriix
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

check ziplist len to avoid crash on empty ziplist convert

Browse Source
This commit is contained in:
Malavan Sotheeswaran 2022-04-05 03:17:41 -07:00 committed by Vivek Saini
parent b73ece90f5
commit 8f02c51f2e
1 changed files with 17 additions and 15 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

30
src/t_zset.cpp
View File

@ -1191,22 +1191,24 @@ void zsetConvert(robj *zobj, int encoding) {
zs->dict = dictCreate(&zsetDictType,NULL); zs->dict = dictCreate(&zsetDictType,NULL);
zs->zsl = zslCreate(); zs->zsl = zslCreate();
eptr = ziplistIndex(zl,0); if (ziplistLen(zl) > 0) {
serverAssertWithInfo(NULL,zobj,eptr != NULL); eptr = ziplistIndex(zl,0);
sptr = ziplistNext(zl,eptr); serverAssertWithInfo(NULL,zobj,eptr != NULL);
serverAssertWithInfo(NULL,zobj,sptr != NULL); sptr = ziplistNext(zl,eptr);
serverAssertWithInfo(NULL,zobj,sptr != NULL);
while (eptr != NULL) { while (eptr != NULL) {
score = zzlGetScore(sptr); score = zzlGetScore(sptr);
serverAssertWithInfo(NULL,zobj,ziplistGet(eptr,&vstr,&vlen,&vlong)); serverAssertWithInfo(NULL,zobj,ziplistGet(eptr,&vstr,&vlen,&vlong));
if (vstr == NULL) if (vstr == NULL)
ele = sdsfromlonglong(vlong); ele = sdsfromlonglong(vlong);
else else
ele = sdsnewlen((char*)vstr,vlen); ele = sdsnewlen((char*)vstr,vlen);
node = zslInsert(zs->zsl,score,ele); node = zslInsert(zs->zsl,score,ele);
serverAssert(dictAdd(zs->dict,ele,&node->score) == DICT_OK); serverAssert(dictAdd(zs->dict,ele,&node->score) == DICT_OK);
zzlNext(zl,&eptr,&sptr); zzlNext(zl,&eptr,&sptr);
}
} }
zfree(zobj->m_ptr); zfree(zobj->m_ptr);