OTA: STM32H7 add check on fopen failures

pennam · andreagilardoni · web-flow · commit 1ac0098fd818 · 2024-06-10T10:02:41.000+02:00
Co-authored-by: Andrea Gilardoni &lt;4046444+andreagilardoni@users.noreply.github.com&gt;
diff --git a/src/ota/implementation/OTASTM32H7.cpp b/src/ota/implementation/OTASTM32H7.cpp
@@ -62,6 +62,9 @@ OTACloudProcessInterface::State STM32H7OTACloudProcess::startOTA() {
 
   decompressed = fopen(_filename.c_str(), "wb");
 
+  if(decompressed == nullptr) {
+    return ErrorOpenUpdateFileFail;
+  }
   // start the download if the setup for ota storage is successful
   return OTADefaultCloudProcessInterface::startOTA();
 }

Original file line number	Diff line number	Diff line change
`@@ -62,6 +62,9 @@ OTACloudProcessInterface::State STM32H7OTACloudProcess::startOTA() {`
`62`	`62`
`63`	`63`	`decompressed = fopen(_filename.c_str(), "wb");`
`64`	`64`
	`65`	`+ if(decompressed == nullptr) {`
	`66`	`+ return ErrorOpenUpdateFileFail;`
	`67`	`+ }`
`65`	`68`	`// start the download if the setup for ota storage is successful`
`66`	`69`	`return OTADefaultCloudProcessInterface::startOTA();`
`67`	`70`	`}`