On 4/9/21 1:28 PM, Simon Horman wrote: > 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]>
Thanks! Applied to master and backported down to 2.13. Best regards, Ilya Maximets. _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
