问题
在C++函数实现时,定义一个函数如下:
bool MCUSerialImpl::InitDevInfo()
{
_devInfo = std::make_shared<MCUInfo>();
if(!_devInfo)
{
LOG_ERROR << "_mcuInfo is null";
return false;
}
Wrapper::TryClearCache<std::string>(_version);
ModelEnv modelEnv = DConfig::Instance().GetModelEnv();
switch(modelEnv)
{
case ModelEnv::SV21Model:
{
_devInfo->_bizDevModel = BizDeviceModel::GD32E230;
_devInfo->_deviceID = "GD32E230";
}
break;
default:
LOG_ERROR << "undefine model.";
assert(false);
}
_devInfo->_version = GetMCUVersion();
}
由于忘记return,调用时出现一连串错误日志打印:
按照原本要走的case是case ModelEnv::SV21Model:但不知道为何走到default这里病狂打印信息,一秒钟几十条的那种,令人抓狂。虽然编译器也有warning,但是编译能通过,最后运行阶段出现深坑,这让人不得不提防warning的地方。
一、分析原因
参考函数调用的堆栈分析:
C函数调用过程原理及函数栈帧分析
ret的指令相当于pop+jum,那么没有return,将出现未定义。
参考此文档:https://blog.csdn.net/jmh1996/article/details/78384083
return 的返回值会存到eax寄存器中,如果没有return,那么eax的值可能是随机的,未定义的。
https://www.cnblogs.com/fzhe/archive/2012/12/13/return.html 参考此文档:https://blog.csdn.net/luoshabugui/article/details/121429487?spm=1001.2101.3001.6650.6&utm_medium=distribute.pc_relevant.none-task-blog-2%7Edefault%7EOPENSEARCH%7ERate-6.pc_relevant_default&depth_1-utm_source=distribute.pc_relevant.none-task-blog-2%7Edefault%7EOPENSEARCH%7ERate-6.pc_relevant_default&utm_relevant_index=9
以上显示结果:定义了返回值,缺少正常retrurn,将在运行时出现未定义的bug。
对应策略:应该在函数中预添加return;void或有返回值都可以预先添加,报错时再修改,以防万一。
|