search

baidu / sofa-pbrpc

A light-weight RPC implement of google protobuf RPC framework.
Other
2.13k stars 655 forks source link

检查 NULL 返回值 #184

Closed viennadd closed 7 years ago

viennadd commented 7 years ago

我们工具扫出 pbjson.cc#L238 parse_msg 有多个返回 NULL 的语句，但函数pb2json 包装 parse_msg 时不对返回做检查就传入 json2string。然后对 parse_msg 返回的 *json 做解引用，希望查询一下

void pb2json(const Message* msg, std::string& str)
{
    rapidjson::Value::AllocatorType allocator;
    rapidjson::Value* json = parse_msg(msg, allocator);
    json2string(json, str);
    delete json;
}
cyshi commented 7 years ago

加上了 多谢 #185