commit 25e20bd8cc614edb6bc3a335d6b4eb2e9df5c054
author Andrew Jeffery <andrew@aj.id.au> Tue Mar 15 22:26:04 2022 +1030
committer Andrew Jeffery <andrew@aj.id.au> Thu Mar 24 23:18:13 2022 +0000
tree 1cfaf27df3fee535c5bda2415eeba8cf337d7861
parent 8418975235cb792235e7cdc8fb4951be2167ced9

NVMeBasicContext: Reject bad I2C bus IDs

We can't possibly query such sensors, so ensure they can't exist.

Signed-off-by: Andrew Jeffery <andrew@aj.id.au>
Change-Id: I899979db57a84272be9b537d87e54756296695cf

src/NVMeSensor.cpp

diff --git a/src/NVMeSensor.cpp b/src/NVMeSensor.cpp
index 89b21b0..d293737 100644
--- a/src/NVMeSensor.cpp
+++ b/src/NVMeSensor.cpp
@@ -33,6 +33,11 @@
            PowerState::on),
     bus(busNumber), objServer(objectServer)
 {
+    if (bus < 0)
+    {
+        throw std::invalid_argument("Invalid bus: Bus ID must not be negative");
+    }
+
     sensorInterface = objectServer.add_interface(
         "/xyz/openbmc_project/sensors/temperature/" + name,
         "xyz.openbmc_project.Sensor.Value");