From: Eric Dumazet <eduma...@google.com>
Date: Tue, 6 Nov 2018 14:51:15 -0800

> On Tue, Nov 6, 2018 at 2:41 PM David Miller <da...@davemloft.net> wrote:
>>
>> From: Eric Dumazet <eduma...@google.com>
>> Date: Tue,  6 Nov 2018 14:25:52 -0800
>>
>> > +             if (unlikely(nlen > sizeof(struct ipv6hdr))) {
>> > +                     if (memcmp(iph + 1, iph2 + 1,
>> > +                                nlen - sizeof(struct ipv6hdr)))
>> > +                             goto not_same_flow;
>> > +             }
>>
>> Is this even possible?
> 
> I believe that nlen can be indeed > sizeof(struct ipv6hdr) in presence
> of exthdrs,
> eg if ipv6_gso_pull_exthdrs() had to be called (line 201)
> 
>  I admit I have not checked if this was actually possible.

Indeed, that does make it possible.

Patch applied, thanks!

Reply via email to