Hi Stephen, Found a compilation issue, I made following change to make it work, let me know if you have any questions: - if (memcmp(&ptp_hdr->master_clock_id, + if (memcmp(&ptp_data->master_clock_id, Regards, Xueming ________________________________ From: Xueming Li Sent: Saturday, April 5, 2025 3:38 PM To: Stephen Hemminger; stable@dpdk.org Subject: Re: [PATCH 23.11] examples/ptpclient: fix message parsing Thanks, patch queued to 23.11.4 release staging Regards, Xueming ________________________________ From: Stephen Hemminger Sent: Thursday, February 20, 2025 1:16 AM To: stable@dpdk.org Cc: Stephen Hemminger Subject: [PATCH 23.11] examples/ptpclient: fix message parsing [ upstream commit 19630bd0d735badb06143086d4f1c50d726b7bad ] Calling memcmp on same structure will always be true. Replace with same conditional used elsewhere. Link: https://pvs-studio.com/en/blog/posts/cpp/1183/ Fixes: ab129e9065a5 ("examples/ptpclient: add minimal PTP client") Signed-off-by: Stephen Hemminger --- examples/ptpclient/ptpclient.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/examples/ptpclient/ptpclient.c b/examples/ptpclient/ptpclient.c index cdf2da64df..92b54f240e 100644 --- a/examples/ptpclient/ptpclient.c +++ b/examples/ptpclient/ptpclient.c @@ -365,7 +365,7 @@ parse_sync(struct ptpv2_data_slave_ordinary *ptp_data, uint16_t rx_tstamp_idx) ptp_data->ptpset = 1; } - if (memcmp(&ptp_hdr->source_port_id.clock_id, + if (memcmp(&ptp_hdr->master_clock_id, &ptp_hdr->source_port_id.clock_id, sizeof(struct clock_id)) == 0) { -- 2.47.2