On 1/21/22 02:12, Jeffrey Walton wrote: > Hi Everyone, > Hi Jeff,
> I am testing Open vSwitch 2.16.2. Undefined Behavior sanitizer is > producing some findings. > > My question is, is the undefined behavior something the project would > be interested in fixing? > I think so. > If so, I can send over the findings and a proposed patch. If not, I > can carry the patch privately. > I had posted a patch series to take care of the issues currently reported by UB Sanitizer: https://patchwork.ozlabs.org/project/openvswitch/list/?series=277900&state=* It will need a new revision and Adrian (in cc) is also working on some more patches to deal with other UB (not flagged by UB Sanitizer). Would the patch you were planning to propose overlap with this work? In any case, I think it would be great if we could work together (reviews and patches) to get tests passing with UB Sanitizer and ideally running these kinds of checks automatically, in CI. E.g., with the series above applied all unit tests are also run with UB Sanitizer enabled: https://github.com/dceara/ovs/runs/4593681037?check_suite_focus=true > Jeff Regards, Dumitru _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
