scott/InfiniTime
1
0
Fork 0
Code Issues Pull requests Actions Packages Projects Releases 1 Wiki Activity

Simple Weather Service : fix out of bounds access while creating Forecast instance.

Browse source
This commit is contained in:
Jean-François Milants 2023-12-10 11:13:18 +01:00 committed by JF
parent 50c679023f
commit fe4b07c610
2 changed files with 9 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
src/components/ble/SimpleWeatherService.cpp
View file

@ -19,6 +19,8 @@
#include "SimpleWeatherService.h" #include "SimpleWeatherService.h"
#include <cstring> #include <cstring>
#include <nrf_log.h> #include <nrf_log.h>
#include <array>
using namespace Pinetime::Controllers; using namespace Pinetime::Controllers;
namespace { namespace {
@ -42,8 +44,10 @@ namespace {
uint64_t timestamp = static_cast<uint64_t>(dataBuffer[2] + (dataBuffer[3] << 8) + (dataBuffer[4] << 16) + (dataBuffer[5] << 24) + uint64_t timestamp = static_cast<uint64_t>(dataBuffer[2] + (dataBuffer[3] << 8) + (dataBuffer[4] << 16) + (dataBuffer[5] << 24) +
((uint64_t) dataBuffer[6] << 32) + ((uint64_t) dataBuffer[7] << 40) + ((uint64_t) dataBuffer[6] << 32) + ((uint64_t) dataBuffer[7] << 40) +
((uint64_t) dataBuffer[8] << 48) + ((uint64_t) dataBuffer[9] << 54)); ((uint64_t) dataBuffer[8] << 48) + ((uint64_t) dataBuffer[9] << 54));
uint8_t nbDays = dataBuffer[10];
std::array<SimpleWeatherService::Forecast::Day, 5> days; std::array<SimpleWeatherService::Forecast::Day, SimpleWeatherService::MaxNbForecastDays> days;
const uint8_t nbDaysInBuffer = dataBuffer[10];
const uint8_t nbDays = std::min(SimpleWeatherService::MaxNbForecastDays, nbDaysInBuffer);
for (int i = 0; i < nbDays; i++) { for (int i = 0; i < nbDays; i++) {
days[i] = SimpleWeatherService::Forecast::Day {dataBuffer[11 + (i * 3)], dataBuffer[12 + (i * 3)], dataBuffer[13 + (i * 3)]}; days[i] = SimpleWeatherService::Forecast::Day {dataBuffer[11 + (i * 3)], dataBuffer[12 + (i * 3)], dataBuffer[13 + (i * 3)]};
} }

4
src/components/ble/SimpleWeatherService.h
View file

@ -46,6 +46,8 @@ namespace Pinetime {
int OnCommand(struct ble_gatt_access_ctxt* ctxt); int OnCommand(struct ble_gatt_access_ctxt* ctxt);
static constexpr uint8_t MaxNbForecastDays = 5;
enum class Icons : uint8_t { enum class Icons : uint8_t {
Sun = 0, // ClearSky Sun = 0, // ClearSky
CloudsSun = 1, // FewClouds CloudsSun = 1, // FewClouds
@ -96,7 +98,7 @@ namespace Pinetime {
uint8_t iconId; uint8_t iconId;
}; };
std::array<Day, 5> days; std::array<Day, MaxNbForecastDays> days;
}; };
std::optional<CurrentWeather> Current() const; std::optional<CurrentWeather> Current() const;