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

More sanity checking; if we get a good header followed immediately by a bad

Browse Source 
one, then the header is no longer valid.
This commit is contained in:
David Given
2019-02-23 20:50:27 +01:00
parent 649c571f81
commit 5b519ba675
1 changed files with 5 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
lib/brother/decoder.cc
View File

@@ -69,19 +69,19 @@ SectorVector BrotherDecoder::decodeToSectors(const RawRecordVector& rawRecords,
{
case BROTHER_SECTOR_RECORD:
{
headerIsValid = false;
if (rawrecord->data.size() < (32+32))
{
headerIsValid = false;
break;
}
nextTrack = decode_header_gcr(read_be16(toBytes(ii+32, ii+48)));
nextSector = decode_header_gcr(read_be16(toBytes(ii+48, ii+64)));
/* Sanity check the sector; sector IDs above 11 can't exist but appear
* occasionally due to read bit errors. */
/* Sanity check the values read; there's no header checksum and
* occasionally we get garbage due to bit errors. */
if (nextSector > 11)
break;
if (nextTrack > 79)
break;
headerIsValid = true;
break;