Team2890/allwpilib
4
0
Fork 0
mirror of https://github.com/wpilibsuite/allwpilib synced 2026-06-20 00:51:42 +00:00
Code Issues Packages Projects Releases Wiki Activity

[wpiutil] DataLog: Fix UB in AppendImpl (#6088)

Browse Source
This commit is contained in:
Peter Johnson
2023-12-23 13:39:39 -08:00
committed by GitHub
parent ab309e34ef
commit c29e8c66cf
1 changed files with 4 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
wpiutil/src/main/native/cpp/DataLog.cpp
View File

@@ -743,8 +743,10 @@ void DataLog::AppendImpl(std::span<const uint8_t> data) {
std::memcpy(buf, data.data(), kBlockSize);
data = data.subspan(kBlockSize);
}
uint8_t* buf = Reserve(data.size());
std::memcpy(buf, data.data(), data.size());
if (!data.empty()) {
uint8_t* buf = Reserve(data.size());
std::memcpy(buf, data.data(), data.size());
}
}
void DataLog::AppendStringImpl(std::string_view str) {