netdev - Re: [PATCH net] net: ethernet: ti: am65-cpts: Fix PTPv1 message type on TX packets

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <4a92f794480b12c21eaeeeb66521dbe978f08414.camel@redhat.com>
Date: Tue, 23 Apr 2024 12:01:01 +0200
From: Paolo Abeni <pabeni@...hat.com>
To: Ravi Gunasekaran <r-gunasekaran@...com>, s-vadapalli@...com, 
	rogerq@...nel.org
Cc: davem@...emloft.net, edumazet@...gle.com, kuba@...nel.org, 
	richardcochran@...il.com, jreeder@...com, netdev@...r.kernel.org, 
	linux-kernel@...r.kernel.org, srk@...com
Subject: Re: [PATCH net] net: ethernet: ti: am65-cpts: Fix PTPv1 message
 type on TX packets

On Fri, 2024-04-19 at 13:35 +0530, Ravi Gunasekaran wrote:
> From: Jason Reeder <jreeder@...com>
> 
> The CPTS, by design, captures the messageType (Sync, Delay_Req, etc.)
> field from the second nibble of the PTP header which is defined in the
> PTPv2 (1588-2008) specification. In the PTPv1 (1588-2002) specification
> the first two bytes of the PTP header are defined as the versionType
> which is always 0x0001. This means that any PTPv1 packets that are
> tagged for TX timestamping by the CPTS will have their messageType set
> to 0x0 which corresponds to a Sync message type. This causes issues
> when a PTPv1 stack is expecting a Delay_Req (messageType: 0x1)
> timestamp that never appears.
> 
> Fix this by checking if the ptp_class of the timestamped TX packet is
> PTP_CLASS_V1 and then matching the PTP sequence ID to the stored
> sequence ID in the skb->cb data structure. If the sequence IDs match
> and the packet is of type PTPv1 then there is a chance that the
> messageType has been incorrectly stored by the CPTS so overwrite the
> messageType stored by the CPTS with the messageType from the skb->cb
> data structure. This allows the PTPv1 stack to receive TX timestamps
> for Delay_Req packets which are necessary to lock onto a PTP Leader.
> 
> Signed-off-by: Jason Reeder <jreeder@...com>
> Signed-off-by: Ravi Gunasekaran <r-gunasekaran@...com>

Please provide a suitable fixes tag, thanks!

Paolo