Hi, On Fri, Nov 13, 2020 at 5:58 PM Alexander Aring <[email protected]> wrote: > ... > + > + p = (union dlm_packet *)((unsigned char *)p->opts.o_opts + > ret); > +
actually this needs to be: p = (union dlm_packet *)((unsigned char *)p->opts.o_opts + le16_to_cpu(p->opts.o_optlen)); otherwise it would break if we add options because we would not "jump over" the header. - Alex
