commit 2fe701225380681d161fb32e233e8388af90c0d1
author Andrew Jeffery <andrew@aj.id.au> Thu Apr 13 23:21:31 2023 +0930
committer Andrew Jeffery <andrew@aj.id.au> Fri Apr 21 14:29:58 2023 +0930
tree b53f7284b58c28dc39ca2c3249d3131cda2eb600
parent 99c03f39fb7d7ebe5b3e1a990c29ec9b2c4fd6a6

platform: pldm_msgbuf for decode_pldm_pdr_repository_chg_event_data()

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

src/platform.c

diff --git a/src/platform.c b/src/platform.c
index c71e42e..fc9b6d3 100644
--- a/src/platform.c
+++ b/src/platform.c
@@ -1703,26 +1703,28 @@
 					      uint8_t *number_of_change_records,
 					      size_t *change_record_data_offset)
 {
-	if (event_data == NULL || event_data_format == NULL ||
-	    number_of_change_records == NULL ||
+	struct pldm_msgbuf _buf;
+	struct pldm_msgbuf *buf = &_buf;
+	int rc;
+
+	if (event_data_format == NULL || number_of_change_records == NULL ||
 	    change_record_data_offset == NULL) {
 		return PLDM_ERROR_INVALID_DATA;
 	}
-	if (event_data_size < PLDM_PDR_REPOSITORY_CHG_EVENT_MIN_LENGTH) {
-		return PLDM_ERROR_INVALID_LENGTH;
+
+	rc = pldm_msgbuf_init(buf, PLDM_PDR_REPOSITORY_CHG_EVENT_MIN_LENGTH,
+			      event_data, event_data_size);
+	if (rc) {
+		return rc;
 	}
 
-	struct pldm_pdr_repository_chg_event_data
-	    *pdr_repository_chg_event_data =
-		(struct pldm_pdr_repository_chg_event_data *)event_data;
+	pldm_msgbuf_extract(buf, event_data_format);
+	pldm_msgbuf_extract(buf, number_of_change_records);
 
-	*event_data_format = pdr_repository_chg_event_data->event_data_format;
-	*number_of_change_records =
-	    pdr_repository_chg_event_data->number_of_change_records;
 	*change_record_data_offset =
 	    sizeof(*event_data_format) + sizeof(*number_of_change_records);
 
-	return PLDM_SUCCESS;
+	return pldm_msgbuf_destroy(buf);
 }
 
 int decode_pldm_pdr_repository_change_record_data(