From 251dddc42d038763ffa0c90aa767dc1c9f65def3 Mon Sep 17 00:00:00 2001 From: Robin Mueller Date: Tue, 28 Feb 2023 01:53:48 +0100 Subject: [PATCH] compile warning fix and defensive programming --- bsp_q7s/core/CoreController.cpp | 3 +++ mission/tmtc/PersistentTmStore.cpp | 4 ++-- 2 files changed, 5 insertions(+), 2 deletions(-) diff --git a/bsp_q7s/core/CoreController.cpp b/bsp_q7s/core/CoreController.cpp index a04ef355..e57b3638 100644 --- a/bsp_q7s/core/CoreController.cpp +++ b/bsp_q7s/core/CoreController.cpp @@ -1224,6 +1224,9 @@ ReturnValue_t CoreController::handleProtInfoUpdateLine(std::string nextLine) { } } wordIdx++; + if(wordIdx >= 10) { + break; + } } return returnvalue::OK; } diff --git a/mission/tmtc/PersistentTmStore.cpp b/mission/tmtc/PersistentTmStore.cpp index 6ceed609..2e7da6f6 100644 --- a/mission/tmtc/PersistentTmStore.cpp +++ b/mission/tmtc/PersistentTmStore.cpp @@ -91,8 +91,8 @@ ReturnValue_t PersistentTmStore::handleCommandQueue(StorageManagerIF& ipcStore, if (accessor.second.size() < 8) { return returnvalue::FAILED; } - uint32_t dumpFromUnixSeconds; - uint32_t dumpUntilUnixSeconds; + uint32_t dumpFromUnixSeconds = 0; + uint32_t dumpUntilUnixSeconds = 0; size_t size = 8; SerializeAdapter::deSerialize(&dumpFromUnixSeconds, accessor.second.data(), &size, SerializeIF::Endianness::NETWORK);