commit f75aca605cfb127a6aa6faf6a8baa43e9cc91ab1
author Andrew Jeffery <andrew@aj.id.au> Thu Apr 13 11:27:07 2023 +0930
committer Andrew Jeffery <andrew@aj.id.au> Fri Apr 21 14:29:58 2023 +0930
tree 1688e802b7e4d850d4341c41a203d20ae0486d01
parent b0e22a7f59de1c625086dc87bb88f2a322a65a92

platform: pldm_msgbuf for decode_get_state_sensor_readings_req()

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

src/platform.c

diff --git a/src/platform.c b/src/platform.c
index 9b750e1..6b967d8 100644
--- a/src/platform.c
+++ b/src/platform.c
@@ -800,23 +800,25 @@
 					 bitfield8_t *sensor_rearm,
 					 uint8_t *reserved)
 {
+	struct pldm_msgbuf _buf;
+	struct pldm_msgbuf *buf = &_buf;
+	int rc;
+
 	if (msg == NULL || sensor_id == NULL || sensor_rearm == NULL) {
 		return PLDM_ERROR_INVALID_DATA;
 	}
 
-	if (payload_length != PLDM_GET_STATE_SENSOR_READINGS_REQ_BYTES) {
-		return PLDM_ERROR_INVALID_LENGTH;
+	rc = pldm_msgbuf_init(buf, PLDM_GET_STATE_SENSOR_READINGS_REQ_BYTES,
+			      msg->payload, payload_length);
+	if (rc) {
+		return rc;
 	}
 
-	struct pldm_get_state_sensor_readings_req *request =
-	    (struct pldm_get_state_sensor_readings_req *)msg->payload;
+	pldm_msgbuf_extract(buf, sensor_id);
+	pldm_msgbuf_extract(buf, &sensor_rearm->byte);
+	pldm_msgbuf_extract(buf, reserved);
 
-	*sensor_id = le16toh(request->sensor_id);
-	*reserved = request->reserved;
-	memcpy(&(sensor_rearm->byte), &(request->sensor_rearm.byte),
-	       sizeof(request->sensor_rearm.byte));
-
-	return PLDM_SUCCESS;
+	return pldm_msgbuf_destroy(buf);
 }
 
 int encode_sensor_event_data(