[Swan-dev] clarify ikev2_ts_r_desc length field

[Andrew Cagney]

On Mon, 2 Mar 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".

Since the RFC calls it:
    Selector Length
baring objections I'll change it to that but lower-case.

> 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