On 1/15/26 11:37 PM, Dima Chumak via dev wrote: > Address issues reported by Coverity after merging > "ovs-router: Multi-table routing infrastructure." series. > > Dima Chumak (2): > ovs-router: Fix JSON object leak in ovs/router/rule/show command. > ovs-router: Fix potential integer overflow. > > lib/ovs-router.c | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) >
Thanks for the fixes! Applied. I also noticed that the rule/show json command prints the 'user' field as integer instead of a boolean. Could you fix that as well? We should add a test for the json variant of this command along with the fix so it's easier to catch these issues. The lack of testing is why CI didn't catch the leak in the first place. Best regards, Ilya Maximets. _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
