commit ff336997185680ad9cde52c76a096f307046bfcd
author Josh Lehan <krellan@google.com> Wed Feb 26 11:13:19 2020 -0800
committer Josh Lehan <krellan@google.com> Wed Feb 26 11:21:59 2020 -0800
tree d4e2afabb13dd70f3763a85db578d781c70311e0
parent 241356e3dcff3e91393c858256ac29d003e6179e

ExitAirTempSensor: Fix potential nullptr dereference

This was found by a compiler warning

Tested: Not tested, but warning no longer appears

Change-Id: Ibe6c3da438c7c622f12606c7d0ae8f4d482ff39a
Signed-off-by: Josh Lehan <krellan@google.com>

src/ExitAirTempSensor.cpp

diff --git a/src/ExitAirTempSensor.cpp b/src/ExitAirTempSensor.cpp
index adf670e..9ba311b 100644
--- a/src/ExitAirTempSensor.cpp
+++ b/src/ExitAirTempSensor.cpp
@@ -228,7 +228,7 @@
             {
 
                 auto cfm = std::get_if<double>(&cfmVariant);
-                if (cfm != nullptr || *cfm >= minSystemCfm)
+                if (cfm != nullptr && *cfm >= minSystemCfm)
                 {
                     maxRpm = self->getMaxRpm(*cfm);
                 }