salfter/fluxengine
1
0
Fork 0
mirror of https://github.com/davidgiven/fluxengine.git synced 2025-10-31 11:17:01 -07:00
Code Issues Packages Projects Releases Wiki Activity

Don't crash when given a missing sector.

Browse Source
This commit is contained in:
David Given
2021-08-01 13:56:51 +02:00
parent 516d43d7a8
commit 44c51f1246
1 changed files with 3 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
arch/northstar/encoder.cc
View File

@@ -131,11 +131,10 @@ public:
int bitsPerRevolution = 100000;
double clockRateUs = 4.00;
if ((physicalTrack < 0) || (physicalTrack >= 35))
if ((physicalTrack < 0) || (physicalTrack >= 35) || sectors.empty())
return std::unique_ptr<Fluxmap>();
const auto& sector = image.get(physicalTrack, physicalSide, 0);
const auto& sector = *sectors.begin();
if (sector->data.size() == NORTHSTAR_PAYLOAD_SIZE_SD) {
bitsPerRevolution /= 2; // FM
} else {
@@ -145,11 +144,8 @@ public:
std::vector<bool> bits(bitsPerRevolution);
unsigned cursor = 0;
for (int sectorId = 0; sectorId < 10; sectorId++)
{
const auto& sectorData = image.get(physicalTrack, physicalSide, sectorId);
for (const auto& sectorData : sectors)
write_sector(bits, cursor, sectorData);
}
if (cursor > bits.size())
Error() << "track data overrun";