Hi Ben, Thanks for the review. I think this bug got introduced when addressing review comments. I have fixed it and sent out a v4.
Regards, Anand Kumar On 9/18/18, 1:28 AM, "Ben Pfaff" <[email protected]> wrote: I didn't really review this but a glance at it showed one possible issue. Missing () around definition here: > +#define CT_MAX_ZONE UINT16_MAX + 1 Bang? > + zoneInfo = OvsAllocateMemoryWithTag(sizeof(OVS_CT_ZONE_INFO) * > + CT_MAX_ZONE, OVS_CT_POOL_TAG); Thanks, Ben. _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
