On Fri, Dec 23, 2016 at 10:31:19AM +0100, Eelco Chaudron wrote: > On 22/12/16 17:14, Ben Pfaff wrote: > >On Wed, Dec 14, 2016 at 07:08:27PM +0100, Eelco Chaudron wrote: > >>When IGMP or MLD packets arrive their content is used without the checksum > >>being verified. With this change the checksum is verified, and the packet > >>is not used for multicast snooping on failure. > >> > >>Signed-off-by: Eelco Chaudron <echau...@redhat.com> > >Thanks for the patch! I applied it to master. I folded in the > >following simplification, which I believe to be correct and also passes > >the test. Please let me know if you see a problem. > Thanks, your change is fine. I just mimicked the code above my change.
Oh, I hadn't noticed that. I think it's best to adjust that code too. I sent out a patch for review. > >Also, I believe that this is your first contribution to Open vSwitch. > >Thank you for helping, and welcome to the team! > Yes this is my first patch, but I'm planning on doing a lot more next year! Great! _______________________________________________ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev
