From a3edfc39a77445f18c69347dc51f8e7c722fa6e5 Mon Sep 17 00:00:00 2001 From: Ansis Atteka Date: Fri, 6 Apr 2012 17:35:12 -0700 Subject: [PATCH] datapath: Do not send notification if ovs_vport_set_options() failed There is no need to send a notification if ovs_vport_set_options() failed and ovs_vport_cmd_set() did not change anything. Issue#10285 Acked-by: Jesse Gross Signed-off-by: Ansis Atteka --- datapath/datapath.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/datapath/datapath.c b/datapath/datapath.c index e8667ad0c..8c4ade97b 100644 --- a/datapath/datapath.c +++ b/datapath/datapath.c @@ -1836,6 +1836,8 @@ static int ovs_vport_cmd_set(struct sk_buff *skb, struct genl_info *info) err = ovs_vport_set_options(vport, a[OVS_VPORT_ATTR_OPTIONS]); if (!err) err = change_vport(vport, a); + else + goto exit_unlock; if (!err && a[OVS_VPORT_ATTR_UPCALL_PID]) vport->upcall_pid = nla_get_u32(a[OVS_VPORT_ATTR_UPCALL_PID]); -- 2.47.0