On 1/29/25 9:20 AM, Frode Nordahl wrote: > When a route has no gateway/next hop IP address, the next hop > register is set to ipN.dst. > > The commit in the fixes tag introduced a logic bug where it would > be possible to produce logical flows where a 32-bit IPv4 address > would be attempted stored in a 128-bit register and vice versa, > producing errors like: > > lflow|WARN|error parsing actions "ip.ttl--; reg8[0..15] = 0; \ > xxreg0 = ip4.dst; xxreg1 = fe80::d3:79ff:fefb:688f; \ > eth.src = 02:d3:79:fb:68:8f; \ > outport = "lrp-XXX"; flags.loopback = 1; reg9[9] = 0; next;": \ > Can't assign 32-bit value to 128-bit destination. > > Cc: Felix Huettner <[email protected]> > Cc: Martin Kalcok <[email protected]> > Fixes: 559924291c7f ("northd: Handle routing for other address families.") > Signed-off-by: Frode Nordahl <[email protected]> > ---
Nice catch, thanks for the bug fix, Frode! Applied to main. Regards, Dumitru _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
