another small logic fix
Some checks are pending
EIVE/eive-obsw/pipeline/head Build queued...
EIVE/eive-obsw/pipeline/pr-v2.1.0-dev This commit looks good

This commit is contained in:
Robin Müller 2023-05-25 08:48:03 +02:00
parent 8d041c6753
commit e1aff4c641
Signed by: muellerr
GPG Key ID: A649FB78196E3849
2 changed files with 2 additions and 1 deletions

View File

@ -82,6 +82,7 @@ will consitute of a breaking change warranting a new major release:
- HK TM store: The HK store dump success event was triggered for cancelled HK dumps. - HK TM store: The HK store dump success event was triggered for cancelled HK dumps.
- When a PUS parsing error occured while parsing a TM store file, the dump completion procedure - When a PUS parsing error occured while parsing a TM store file, the dump completion procedure
was always executed. was always executed.
- Some smaller logic fixes in the TM store base class
# [v2.0.5] 2023-05-11 # [v2.0.5] 2023-05-11

View File

@ -256,6 +256,7 @@ ReturnValue_t PersistentTmStore::getNextDumpPacket(PusTmReader& reader, bool& fi
if (state == State::IDLE or dumpParams.pendingPacketDump) { if (state == State::IDLE or dumpParams.pendingPacketDump) {
return returnvalue::FAILED; return returnvalue::FAILED;
} }
fileHasSwapped = false;
reader.setReadOnlyData(fileBuf.data() + dumpParams.currentSize, reader.setReadOnlyData(fileBuf.data() + dumpParams.currentSize,
fileBuf.size() - dumpParams.currentSize); fileBuf.size() - dumpParams.currentSize);
// CRC check to fully ensure this is a valid TM // CRC check to fully ensure this is a valid TM
@ -270,7 +271,6 @@ ReturnValue_t PersistentTmStore::getNextDumpPacket(PusTmReader& reader, bool& fi
fileHasSwapped = true; fileHasSwapped = true;
return loadNextDumpFile(); return loadNextDumpFile();
} }
fileHasSwapped = false;
dumpParams.pendingPacketDump = true; dumpParams.pendingPacketDump = true;
return returnvalue::OK; return returnvalue::OK;
} }