On 2026-01-16 7:21 PM, Ilya Maximets wrote: > External email: Use caution opening links or attachments > > > 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. >
Thanks! > 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. > A new fix has been submitted. Regards, Dima > Best regards, Ilya Maximets. _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
