Faicker Mo <[email protected]> writes: > Fix the coredump of ovs_assert, The backtrace is as follows, > #8 0x0000561ee52084bb in ovs_assert_failure (where=<>, function=<>, > condition=<>) at ../lib/util.c:89 > #9 0x0000561ee50f8ab2 in dp_packet_set_size (b=<>, v=<>) at > ../lib/dp-packet.h:687 > #10 dp_packet_set_size (v=<>, b=0x7f7f88143e80) at ../lib/dp-packet.h:687 > #11 dp_packet_put_uninit (size=395, b=0x7f7f88143e80) at > ../lib/dp-packet.c:355 > #12 dp_packet_put (b=0x7f7f88143e80, p=0x7f7f88143ce2, size=395) at > ../lib/dp-packet.c:376 > #13 0x0000561ee512c147 in ipf_reassemble_v4_frags (ipf_list=0x7f7f88110810) > at ../lib/ipf.c:430 > > The mbuf data_len is a uint16_t field, which includes the ether header. > So does IPv6. > > Fixes: 4ea96698f667 ("Userspace datapath: Add fragmentation handling.") > > Signed-off-by: Faicker Mo <[email protected]> > ---
Thanks Faicker, and Mike. I've applied this and backported down through branch-2.17. On a side note, it seems like Faicker's email address has changed from the AUTHORS.rst file. If you're able and willing, please submit a change to the mailmap and AUTHORS.rst file(s). _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
