hiredis是redis官方提供的c客戶端庫。在讀代碼的過程中，發現了一個bug，記錄一下。
hiredis里定義了一個上下文結構（struct redisContext），代碼如下（deps/hiredis/hiredis.h）：
https://github.com/antirez/hiredis/blob/master/hiredis.h
157
/*
Context for a connection to Redis
*/
158
typedef
struct
redisContext {
159
int
err;
/*
Error flags, 0 when there is no error
*/
160
char
errstr[
128
];
/*
String representation of error when applicable
*/
161
int
fd;
162
int
flags;
163
char
*obuf;
/*
Write buffer
*/
164
redisReader *reader;
/*
Protocol reader
*/
165
} redisContext;
針對這個結構，有一個對應的free function，代碼如下（deps/hiredis/hiredis.c）：
https://github.com/antirez/hiredis/blob/master/hiredis.c
1004
void
redisFree(redisContext *
c) {
1005
if
(c->fd >
0
)
1006
close(c->
fd);
1007
if
(c->obuf !=
NULL)
1008
sdsfree(c->
obuf);
1009
if
(c->reader !=
NULL)
1010
redisReaderFree(c->
reader);
1011
free(c);
1012
}
對照下，可以看到，定義中obuf成員使用的是char* ，而在釋放操作時，卻是按照sds結構（redis自己定義的類string數據結構）進行的釋放。
如果再看下sdsfree的函數（deps/hiredis/sds.c），就能看到可能造成的災難性后果：
https://github.com/antirez/hiredis/blob/master/sds.c
76
void
sdsfree(sds s) {
77
if
(s == NULL)
return
;
78
free(s-
sizeof
(
struct
sdshdr));
79
}
如上。在hiredis的github中，錯誤仍可以看到。
?
剛給redis DB郵件組發了封郵件，不知是否會被回復。
更多文章、技術交流、商務合作、聯系博主
微信掃碼或搜索：z360901061
微信掃一掃加我為好友
QQ號聯系： 360901061
您的支持是博主寫作最大的動力，如果您喜歡我的文章，感覺我的文章對您有幫助，請用微信掃描下面二維碼支持博主2元、5元、10元、20元等您想捐的金額吧，狠狠點擊下面給點支持吧，站長非常感激您！手機微信長按不能支付解決辦法：請將微信支付二維碼保存到相冊，切換到微信，然后點擊微信右上角掃一掃功能，選擇支付二維碼完成支付。
【本文對您有幫助就好】元
