1.CSVData.cpp 里 unserialize 用了 8*2、125*2,但 serialize 只写 8 + 125 字节。
m_svRawData.insert 的 end 指针是 pszBuffer + 125*2,没有用 index 计算,可能把无效区域一起拷进去。
一旦 size 实际是 133(不是 266),就会直接越界,堆会被破坏,m_svDatas.clear() 在销毁元素时崩。
| | |
| | | CSVData svData; |
| | | int nRet = svData.unserialize(&pszData[0], (int)size); |
| | | if (nRet < 0) return nRet; |
| | | Lock(); |
| | | m_svDatas.push_back(svData); |
| | | Unlock(); |
| | | |
| | | if (m_listener.onSVDataReport != nullptr) { |
| | | m_listener.onSVDataReport(this, &svData); |
| | |
| | | |
| | | int CSVData::serialize(char* pszBuffer, int nBufferSize) |
| | | { |
| | | if (nBufferSize < 133) return -1; |
| | | if (nBufferSize < 133 * 2) return -1; |
| | | |
| | | int index = 0; |
| | | CToolUnits::convertString(&pszBuffer[index], 8, m_strTime); |
| | | index += 8; |
| | | CToolUnits::convertString(&pszBuffer[index], 8 * 2, m_strTime); |
| | | index += 8 * 2; |
| | | |
| | | memcpy(&pszBuffer[index], m_svRawData.data(), 125); |
| | | index += 125; |
| | | memcpy(&pszBuffer[index], m_svRawData.data(), 125 * 2); |
| | | index += 125 * 2; |
| | | |
| | | return 133; |
| | | return 133 * 2; |
| | | } |
| | | |
| | | int CSVData::unserialize(const char* pszBuffer, int nBufferSize) |
| | | { |
| | | if (nBufferSize < 133) return -1; |
| | | if (pszBuffer == nullptr) return -1; |
| | | if (nBufferSize < 133 * 2) return -1; |
| | | |
| | | int index = 0; |
| | | CSVData svData; |
| | | CToolUnits::convertString(&pszBuffer[index], 8 * 2, m_strTime); |
| | | index += 8 * 2; |
| | | |
| | | m_svRawData.clear(); |
| | | m_svRawData.insert(m_svRawData.end(), (uint8_t*)(&pszBuffer[index]), (uint8_t*)(pszBuffer)+(125 * 2)); |
| | | if (nBufferSize < index + 125 * 2) return -1; |
| | | m_svRawData.insert( |
| | | m_svRawData.end(), |
| | | (const uint8_t*)&pszBuffer[index], |
| | | (const uint8_t*)&pszBuffer[index + 125 * 2]); |
| | | index += 125 * 2; |
| | | |
| | | return 133; |
| | | return 133 * 2; |
| | | } |
| | | } |