[Swan-dev] clarify ikev2_ts_r_desc length field

Paul Wouters paul at nohats.ca
Mon Mar 2 14:29:45 UTC 2020


I will recheck. I thought this was one exceptional when I looked at it. If not, I will change it back.

Paul

Sent from my iPhone

> On Mar 2, 2020, at 09:22, D. Hugh Redelmeier <hugh at mimosa.com> wrote:
> 
> I don't think that the logging change is good.
> 
> 1. (almost?) all lengths in IKE are the length of the whole structure.
>   By changing logging for only one, you raise more questions than you 
>   answer.  "ft_len" means "length of the whole structure".
> 
> 3. The result is quite wordy.
> 
> I suggest that if there are any lengths that are NOT of the whole 
> structure, the logging should be changed for them.
> 
> | commit 054beecd614803522e62d30213e8be5b78205d30
> | Author: Paul Wouters <pwouters at redhat.com>
> | Date:   Sun Mar 1 22:12:42 2020 -0500
> | 
> |     documentation: clarify ikev2_ts_r_desc length field
> |     
> |     It's not the length of the variable part but of the entire TS structure.
> _______________________________________________
> Swan-dev mailing list
> Swan-dev at lists.libreswan.org
> https://lists.libreswan.org/mailman/listinfo/swan-dev



More information about the Swan-dev mailing list