On 3/6/2019 7:31 AM, Juerg Haefliger wrote:

[snip]


Somehow the patch was shared on three different open source lists (the
bug, linux-kernel, and ovs-dev), yet only one of the three actually
retained the message:

https://mail.openvswitch.org/pipermail/ovs-dev/2018-September/352395.html
Thanks for the link, Jarno, but that patch doesn't help at all.

FWIW, removing the following case statement makes the test pass:

                case OVS_CT_ATTR_EVENTMASK:
                        info->have_eventmask = true;
                        info->eventmask = nla_get_u32(a);
                        break;

But changing it to something like the below also blows up the machine:

                case OVS_CT_ATTR_EVENTMASK:
                        info->have_eventmask = false;
                        info->eventmask = nla_get_u32(a);
                        break;

Also bad:

                case OVS_CT_ATTR_EVENTMASK:
                        break;

I'm not saying these tests make any sense, just saying :-)

...Juerg

Running the test under valgrind makes the bug go away... :(

But that's a clue.

- Greg



_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to