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

Fix several embarassingly stupid bugs in the brother 120kB encoder code path.

Browse Source
This commit is contained in:
David Given
2020-05-14 10:41:25 +02:00
parent 9a5958f80b
commit 6ca06ecafb
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
arch/brother/encoder.cc
View File

@@ -135,7 +135,7 @@ std::unique_ptr<Fluxmap> BrotherEncoder::encode(
switch (_format)
{
case 120:
if ((physicalTrack < 0) || (physicalTrack >= BROTHER_TRACKS_PER_120KB_DISK)
if ((physicalTrack < 0) || (physicalTrack >= (BROTHER_TRACKS_PER_120KB_DISK*2))
|| (physicalTrack & 1))
return std::unique_ptr<Fluxmap>();
logicalTrack = physicalTrack/2;
@@ -164,7 +164,7 @@ std::unique_ptr<Fluxmap> BrotherEncoder::encode(
const auto& sectorData = allSectors.get(logicalTrack, 0, sectorId);
fillBitmapTo(bits, cursor, headerCursor, { true, false });
write_sector_header(bits, cursor, physicalTrack, sectorId);
write_sector_header(bits, cursor, logicalTrack, sectorId);
fillBitmapTo(bits, cursor, dataCursor, { true, false });
write_sector_data(bits, cursor, sectorData->data);
}