tests: modbus: Fix potential memory overwrite (coverity)

Fixes zephyrproject-rtos#33816 Fixes zephyrproject-rtos#33811 Fixes zephyrproject-rtos#33793 Fixes zephyrproject-rtos#33797 Fixes zephyrproject-rtos#33796 Signed-off-by: David Leach <[email protected]>
bibibibi89 · Apr 1, 2021 · 8272b32 · 8272b32
1 parent 3f0da93
commit 8272b32
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/tests/subsys/modbus/src/test_modbus_raw.c b/tests/subsys/modbus/src/test_modbus_raw.c
@@ -26,7 +26,7 @@ int server_raw_cb(const int iface, const struct modbus_adu *adu)
 	tmp_adu.unit_id = adu->unit_id;
 	tmp_adu.fc = adu->fc;
 	memcpy(tmp_adu.data, adu->data,
-	       MIN(adu->length, CONFIG_MODBUS_BUFFER_SIZE));
+	       MIN(adu->length, sizeof(tmp_adu.data)));
 
 	LOG_HEXDUMP_DBG(tmp_adu.data, tmp_adu.length, "resp");
 	k_sem_give(&received);
@@ -51,7 +51,7 @@ int client_raw_cb(const int iface, const struct modbus_adu *adu)
 	tmp_adu.unit_id = adu->unit_id;
 	tmp_adu.fc = adu->fc;
 	memcpy(tmp_adu.data, adu->data,
-	       MIN(adu->length, CONFIG_MODBUS_BUFFER_SIZE));
+	       MIN(adu->length, sizeof(tmp_adu.data)));
 
 	LOG_HEXDUMP_DBG(tmp_adu.data, tmp_adu.length, "c->s");