Check DHT Sensor values for errors (#1392)

* checkval * bump to beta3 * fix and parenthesis
mongoose-os-apps · Jun 10, 2024 · db34ed9 · db34ed9
1 parent 8ccdc21
commit db34ed9
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 4 deletions.
diff --git a/mos.yml b/mos.yml
@@ -1,6 +1,6 @@
 author: Shelly-HomeKit contributors
 description: A HomeKit firmware for Shelly switches
-version: 2.12.0-beta2
+version: 2.12.0-beta3
 
 libs_version: latest
 modules_version: latest

diff --git a/src/DHT/shelly_dht_sensor.cpp b/src/DHT/shelly_dht_sensor.cpp
@@ -83,9 +83,20 @@ StatusOr<float> DHTSensor::GetHumidity() {
 }
 
 void DHTSensor::UpdateTemperatureCB() {
-  // std::nan
-  result_ = mgos_dht_get_temp(dht);
-  result_humidity_ = mgos_dht_get_humidity(dht);
+  float result = mgos_dht_get_temp(dht);
+  float result_humidity = mgos_dht_get_humidity(dht);
+
+  // check values for validity
+  if (((result == 0) and (result_humidity == 0)) or isnan(result) or
+      isnan(result_humidity)) {
+    // error during readout do nothing for now
+    LOG(LL_INFO, ("DHT: invalid value received"));
+    return;
+  }
+
+  result_ = result;
+  result_humidity_ = result_humidity;
+
   if (notifier_) {
     notifier_();
   }