记录一次protobuf 段错误的bug
2020-04-27 本文已影响0人
犭虫彳亍口苗
此bug不具典型性,仅为个人开发笔记。
概况
在项目中使用了protobuf作为数据序列化以及消息体传输的协议。
开发过程中发现一崩溃bug与protobuf有关。
bug表现
简洁描述如下:
- 定义了一个protobuf的message,假设为 P.proto。
syntax = "proto3";
package cicv.routing;
message Waypoint {
string id = 1;
double s = 2;
}
message RoutingRequest {
repeated Waypoint waypoint = 1;
}
}
- 在A.cpp中使用了P.proto中的RoutingRequest。
#include "google/protobuf/util/json_util.h"
#include "RoutingRequest.pb.h"
std::string A:fun{
...
cicv::routing RoutingRequest routing_request;
auto waypoint = routing_request.add_waypoint();
....
std::string json;
google::protobuf::util::MessageToJsonString(routing_request, &json);
return json;
}
bug表现为,在A::fun{}
执行完毕,析构栈对象routing_request
时崩溃。经过跟踪,是在析构其成员变量waypoint
时错误。尝试了很多种写法,总结下来就是,只要在A::fun{}
中调用了routing_request
动态创建对象 的方法,在A::fun{}
结束后析构routing_request
就没办法析构创建的对象。
- 遂尝试了另外一种写法:
新增创建了B.cpp,其中定义了classB。
#include "RoutingRequest.pb.h"
string A::fun{
//改为完全在B中构建json
cicv::routing RoutingRequest routing_request;
B b;
return b.BuildJson(&routing_request);
}
B.cpp
std::string B::BuildJson( cicv::routing RoutingRequest* request){
std::string json;
auto waypoint = routing_request.add_waypoint();
....
google::protobuf::util::MessageToJsonString(routing_request, &json);
return json;
}
也就是改成了在class B中调用add_waypoint()
。神奇的是, 此时A::fun{}
结束后能正常析构栈对象routing_request
了。
解决
- 初步估计是内存错误,经过跟踪,确实发现在析构routing_request时,其中记录的waypoint的地址完全错误。
- 确认protobuf版本对应正确,排除protobuf库版本与proto c++文件不对应的可能。
- 确认正确依赖了 pthread.so。不依赖此库protobuf同样会产生奇怪的问题。
- 确认protobuf库与程序库采用同样的编译工具链以及编译参数。
- 重新构建了一个简单的测试工程尝试以上写法,未发现bug,排除了protobuf使用方式的问题。
- 重新思考bug,浏览了A.cpp 依赖的所有头文件,在其中一个头文件中发现了以下语句
#pragma pack(4)
...
此编译语句为了告诉编译器采用4字节对齐,此语句与内存相关!
- 搜索了pragma pack的详细用法,发现pragma pack应该成对使用,在不需要修改对齐字节后,应取消。
隧在文件末尾添加#pragma pack(),修改回编译器默认的字节对齐!
#pragma pack(4)
...
#pragma pack()
- Bug解决了。。。
总结:
- 尽量不要在代码中涉及编译指令,很不方便问题定位。
- 与内存相关的指令更应注意。