记一hiredis redisCommand坑

最新推荐文章于 2025-11-12 01:29:43 发布

转载最新推荐文章于 2025-11-12 01:29:43 发布 · 2.6k 阅读

redis 专栏收录该内容

23 篇文章

订阅专栏

本文讨论了Redis命令格式化过程中遇到的大小类型转换错误问题，特别是当开发环境的int与size_t类型大小不匹配时，导致的内存溢出问题。通过将所有涉及大小的变量转换为size_t类型，可以有效解决此问题。

实际上应该说是redisvFormatCommand的坑，代码如下：

[cpp]view plaincopyprint? 
   
 case 'b':  
 <span style="white-space:pre">    </span>arg = va_arg(ap,char*);  
         size = va_arg(ap,size_t);  
         if (size > 0)  
            newarg = sdscatlen(curarg,arg,size);  
            break;  

就是%b的时候的格式化命令，调用如下（修正后的）：

[cpp]view plaincopyprint? 
   
 reply = (redisReply*)redisCommand(redis_context, "SET %b%b%b %b",  
                                       g_preffix, (size_t)preffix_len, key, (size_t)key_len, g_suffix, (size_t)suffix_len,  
                                       value, (size_t)value_len);  

调用代码中preffix_len,key_len,suffix_len,value_len都是int类型的数据，我的开发环境为centos 6.6 x64位（如果是32位开发机的话都不会碰到这个问题），在gcc 4.4.7上，我的size_t是64位，int是32位，这样在vFormatCommand中va_arg取出size的时候就会出错了，修正方案也比较简单，如上，在调用代码中，将所有size的变量转为size_t之后再调用。

不修正的话最可能碰到的问题就是Out of memory....