commit 9667f58239302e6053da27876981cdd0bf9c5de2
author Andrew Jeffery <andrew@aj.id.au> Fri Apr 14 00:39:21 2023 +0930
committer Andrew Jeffery <andrew@aj.id.au> Fri Apr 21 14:29:59 2023 +0930
tree 06974c32c6abb46d3f58788692061cfee3491c1c
parent 42dd4e596b43dee4973c7da165f8a06e2d6689ae

platform: pldm_msgbuf for decode_set_event_receiver_req()

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

src/platform.c

diff --git a/src/platform.c b/src/platform.c
index e07c571..35a5a72 100644
--- a/src/platform.c
+++ b/src/platform.c
@@ -2004,23 +2004,31 @@
 				  uint16_t *heartbeat_timer)
 
 {
+	struct pldm_msgbuf _buf;
+	struct pldm_msgbuf *buf = &_buf;
+	int rc;
+
 	if (msg == NULL || event_message_global_enable == NULL ||
 	    transport_protocol_type == NULL ||
 	    event_receiver_address_info == NULL || heartbeat_timer == NULL) {
 		return PLDM_ERROR_INVALID_DATA;
 	}
 
-	if (payload_length != PLDM_SET_EVENT_RECEIVER_REQ_BYTES) {
-		return PLDM_ERROR_INVALID_LENGTH;
+	rc = pldm_msgbuf_init(buf, PLDM_SET_EVENT_RECEIVER_REQ_BYTES,
+			      msg->payload, payload_length);
+	if (rc) {
+		return rc;
 	}
 
-	struct pldm_set_event_receiver_req *request =
-	    (struct pldm_set_event_receiver_req *)msg->payload;
+	pldm_msgbuf_extract(buf, event_message_global_enable);
+	pldm_msgbuf_extract(buf, transport_protocol_type);
+	pldm_msgbuf_extract(buf, event_receiver_address_info);
+	pldm_msgbuf_extract(buf, heartbeat_timer);
 
-	*event_message_global_enable = request->event_message_global_enable,
-	*transport_protocol_type = request->transport_protocol_type,
-	*event_receiver_address_info = request->event_receiver_address_info,
-	*heartbeat_timer = le16toh(request->heartbeat_timer);
+	rc = pldm_msgbuf_destroy(buf);
+	if (rc) {
+		return rc;
+	}
 
 	if ((*event_message_global_enable ==
 	     PLDM_EVENT_MESSAGE_GLOBAL_ENABLE_ASYNC_KEEP_ALIVE) &&