Merge pull request #6739 from trevor211/fixMemoryLeak

Fix potential memory leak of rioWriteBulkStreamID().
This commit is contained in:
Salvatore Sanfilippo
2020-01-13 13:23:49 +01:00
committed by GitHub
3 changed files with 25 additions and 6 deletions

View File

@ -1149,7 +1149,10 @@ int rioWriteBulkStreamID(rio *r,streamID *id) {
int retval;
sds replyid = sdscatfmt(sdsempty(),"%U-%U",id->ms,id->seq);
if ((retval = rioWriteBulkString(r,replyid,sdslen(replyid))) == 0) return 0;
if ((retval = rioWriteBulkString(r,replyid,sdslen(replyid))) == 0) {
sdsfree(replyid);
return 0;
}
sdsfree(replyid);
return retval;
}