Fix weird problem with lock being un-initialized

kivancsikert · Jan 19, 2025 · d53418d · d53418d
1 parent 95237fb
commit d53418d
Showing 1 changed file with 6 additions and 3 deletions.
diff --git a/main/kernel/PowerManager.hpp b/main/kernel/PowerManager.hpp
@@ -17,7 +17,8 @@ namespace farmhub::kernel {
 
 class PowerManagementLock {
 public:
-    PowerManagementLock(const std::string& name, esp_pm_lock_type_t type) : name(name) {
+    PowerManagementLock(const std::string& name, esp_pm_lock_type_t type)
+        : name(name) {
         ESP_ERROR_CHECK(esp_pm_lock_create(type, 0, name.c_str(), &lock));
     }
 
@@ -31,7 +32,7 @@ class PowerManagementLock {
 
 private:
     const std::string name;
-    esp_pm_lock_handle_t lock;
+    esp_pm_lock_handle_t lock = nullptr;
 
     friend class PowerManagementLockGuard;
 };
@@ -44,7 +45,9 @@ class PowerManagementLockGuard {
     }
 
     ~PowerManagementLockGuard() {
-        ESP_ERROR_CHECK(esp_pm_lock_release(lock.lock));
+        if (lock.lock != nullptr) {
+            ESP_ERROR_CHECK(esp_pm_lock_release(lock.lock));
+        }
     }
 
     // Delete copy constructor and assignment operator to prevent copying