🩹 Return 0 for bad index in Temperature::getHeaterPower (MarlinFirmware#27037)

ellensp · thinkyhead · EvilGremlin · commit 676a0649ed27 · 2025-05-15T12:36:43.000+03:00
Co-authored-by: Scott Lahteine &lt;thinkyhead@users.noreply.github.com&gt;
diff --git a/Marlin/src/module/temperature.cpp b/Marlin/src/module/temperature.cpp
@@ -1303,8 +1303,10 @@ int16_t Temperature::getHeaterPower(const heater_id_t heater_id) {
     #if HAS_COOLER
       case H_COOLER: return temp_cooler.soft_pwm_amount;
     #endif
-    default:
-      return TERN0(HAS_HOTEND, temp_hotend[heater_id].soft_pwm_amount);
+    #if HAS_HOTEND
+      case 0 ... HOTENDS - 1: return temp_hotend[heater_id].soft_pwm_amount;
+    #endif
+    default: return 0;
   }
 }
 

Original file line number	Diff line number	Diff line change
`@@ -1303,8 +1303,10 @@ int16_t Temperature::getHeaterPower(const heater_id_t heater_id) {`
`1303`	`1303`	`#if HAS_COOLER`
`1304`	`1304`	`case H_COOLER: return temp_cooler.soft_pwm_amount;`
`1305`	`1305`	`#endif`
`1306`		`- default:`
`1307`		`- return TERN0(HAS_HOTEND, temp_hotend[heater_id].soft_pwm_amount);`
	`1306`	`+ #if HAS_HOTEND`
	`1307`	`+ case 0 ... HOTENDS - 1: return temp_hotend[heater_id].soft_pwm_amount;`
	`1308`	`+ #endif`
	`1309`	`+ default: return 0;`
`1308`	`1310`	`}`
`1309`	`1311`	`}`
`1310`	`1312`