On Mon, Oct 23, 2023 at 04:22:32PM +0200, Eelco Chaudron wrote:
> It appears that an issue existed in the count_common_prefix_run()
> function from the beginning. This problem came to light while
> addressing 'Dead assignment' warnings identified by the Clang
> static analyzer.
>
> Instead of updating the extra_prefix_len with the current (next)
> value, the next value was inadvertently updated with extra_prefix_len.
> This patch rectifies this behavior.
>
> Fixes: 95a5454c5110 ("ofp-print: Abbreviate lists of fields in table features
> output.")
> Signed-off-by: Eelco Chaudron <[email protected]>
Thanks Eelco,
I agree with the reasoning in the commit message.
Reviewed-by: Simon Horman <[email protected]>
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
