diff options
| author | Rich Lane <rlane@bigswitch.com> | 2013-02-08 13:18:01 -0800 |
|---|---|---|
| committer | Jesse Gross <jesse@nicira.com> | 2013-02-22 17:01:57 -0800 |
| commit | cb7c5bdffb727a3d4dea5247d9d1d52238b01d90 (patch) | |
| tree | 95208844ea10a71a12193e1d70fe3754ce68145f /net/openvswitch | |
| parent | 734907e82d21a75a514b80164185427a832a00c0 (diff) | |
openvswitch: Fix ovs_vport_cmd_new return value on success
If the pointer does not represent an error then the PTR_ERR
macro may still return a nonzero value.
Signed-off-by: Rich Lane <rlane@bigswitch.com>
Signed-off-by: Jesse Gross <jesse@nicira.com>
Diffstat (limited to 'net/openvswitch')
| -rw-r--r-- | net/openvswitch/datapath.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/net/openvswitch/datapath.c b/net/openvswitch/datapath.c index 5e275b903f3c..a2cd3e6d03a2 100644 --- a/net/openvswitch/datapath.c +++ b/net/openvswitch/datapath.c @@ -1691,6 +1691,7 @@ static int ovs_vport_cmd_new(struct sk_buff *skb, struct genl_info *info) if (IS_ERR(vport)) goto exit_unlock; + err = 0; reply = ovs_vport_cmd_build_info(vport, info->snd_portid, info->snd_seq, OVS_VPORT_CMD_NEW); if (IS_ERR(reply)) { |