On Thu, Apr 08, 2021 at 09:33:12PM +0800, wangyunjian wrote:
> From: Yunjian Wang <[email protected]>
> 
> Before info.tc_modify_flow_deleted is assigned a value, error
> processing of other statements goes to the out label. In the
> out label, the uninitialized variant is used for condition
> determination, which may cause uncertain behavior.
> 
> Fixes: 65b84d4a32bd ("dpif-netlink: avoid netlink modify flow put op failed 
> after tc modify flow put op failed.")
> Signed-off-by: Mengfan Lv <[email protected]>
> Signed-off-by: Yunjian Wang <[email protected]>

Thanks, I see the problem you describe and agree
it is wise to take the approach of initialising this as early as possible.

Reviewed-by: Simon Horman <[email protected]>
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to