On Tue, Aug 20, 2024 at 09:14:41AM -0400, Mike Pattrick wrote:
> Currently the Clang analyzer will complain about usage of an
> uninitialized variable in the classifier. This is a false positive, but
> not for a reason that could easily be detectable by clang.
> 
> The classifier is not safe for multiple writer threads to use
> simultaniously so all callers protect these functions from simultanious
> writes. However, this is not so clear from the code's static analysis
> alone. To help Clang out here, the n_indicies count is saved onto the
> stack instead of accessed from the subtables struct repeatedly.
> 
> Signed-off-by: Mike Pattrick <[email protected]>

Acked-by: Simon Horman <[email protected]>

_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to