On 16/04/2019 10:12, Florian Westphal wrote:
> On 4/15/2019 5:14 PM, Colin King wrote:
>>> diff --git a/net/netfilter/nf_conntrack_h323_types.c b/net/netfilter/nf_conntrack_h323_types.c
>>> index d880f3523c1d..95a0b3d6b24d 100644
>>> --- a/net/netfilter/nf_conntrack_h323_types.c
>>> +++ b/net/netfilter/nf_conntrack_h323_types.c
>>> @@ -1109,7 +1109,7 @@ static const struct field_t _SecurityCapabilities[] = { /* SEQUENCE */
>>> _NonStandardParameter},
>>> {FNAME("encryption") CHOICE, 2, 3, 3, SKIP | EXT, 0,
>>> _SecurityServiceMode},
>>> - {FNAME("authenticaton") CHOICE, 2, 3, 3, SKIP | EXT, 0,
>>> + {FNAME("authentication") CHOICE, 2, 3, 3, SKIP | EXT, 0,
>>> _SecurityServiceMode},
>>> {FNAME("integrity") CHOICE, 2, 3, 3, SKIP | EXT, 0,
>>> _SecurityServiceMode},
>
> https://www.itu.int/ITU-T/formal-language/itu-t/h/h225-0/2009/H323-MESSAGES.html says:
> ...
> SecurityCapabilities ::= SEQUENCE {
> nonStandard NonStandardParameter OPTIONAL,
> encryption SecurityServiceMode,
> authenticaton SecurityServiceMode,
> integrity SecurityServiceMode,
> ...
> }
>
> so this spelling seems intentional.
>
Ah, thanks for spotting that, my bad.
