Revert "OTA: use sha256 from ArduinoBearSSL"

This reverts commit 51f11c1.

Author: pennam

src/ota/interface/OTAInterface.cpp

@@ -108,11 +108,11 @@ OTACloudProcessInterface::State OTACloudProcessInterface::otaBegin() {
     OtaBeginUpId,
   };
 
-  SHA256Class sha256_calc;
+  SHA256 sha256_calc;
   calculateSHA256(sha256_calc);
 
-  sha256_calc.endHash();
-  sha256_calc.readBytes(msg.params.sha, SHA256_DIGEST_SIZE);
+  sha256_calc.finalize(sha256);
+  memcpy(msg.params.sha, sha256, SHA256::HASH_SIZE);
 
   DEBUG_VERBOSE("calculated SHA256: "
       "0x%02X%02X%02X%02X%02X%02X%02X%02X%02X%02X%02X%02X%02X%02X%02X%02X"
@@ -128,15 +128,15 @@ OTACloudProcessInterface::State OTACloudProcessInterface::otaBegin() {
   return Idle;
 }
 
-void OTACloudProcessInterface::calculateSHA256(SHA256Class& sha256_calc) {
+void OTACloudProcessInterface::calculateSHA256(SHA256& sha256_calc) {
   auto res = appFlashOpen();
   if(!res) {
     // TODO return error
     return;
   }
 
-  sha256_calc.beginHash();
-  sha256_calc.write(
+  sha256_calc.begin();
+  sha256_calc.update(
     reinterpret_cast<const uint8_t*>(appStartAddress()),
     appSize());
   appFlashClose();

src/ota/interface/OTAInterface.h

@@ -18,7 +18,7 @@
 
 #if OTA_ENABLED
 #include "../OTATypes.h"
-#include <SHA256.h>
+#include "tls/utility/SHA256.h"
 
 #include <interfaces/CloudProcess.h>
 #include <Arduino_DebugUtils.h>
@@ -156,10 +156,10 @@ class OTACloudProcessInterface: public CloudProcess {
   virtual bool appFlashClose() = 0;
 
   // sha256 is going to be used in the ota process for validation, avoid calculating it twice
-  uint8_t sha256[SHA256_DIGEST_SIZE];
+  uint8_t sha256[SHA256::HASH_SIZE];
 
   // calculateSHA256 method is overridable for platforms that do not support access through pointer to program memory
-  virtual void calculateSHA256(SHA256Class&); // FIXME return error
+  virtual void calculateSHA256(SHA256&); // FIXME return error
 private:
   void clean();