diff options
| author | Rich Lane <rlane@bigswitch.com> | 2013-02-08 16:18:01 -0500 |
|---|---|---|
| committer | Jesse Gross <jesse@nicira.com> | 2013-02-22 20:01:57 -0500 |
| commit | cb7c5bdffb727a3d4dea5247d9d1d52238b01d90 (patch) | |
| tree | 95208844ea10a71a12193e1d70fe3754ce68145f /net | |
| 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')
| -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) | |||
| 1691 | if (IS_ERR(vport)) | 1691 | if (IS_ERR(vport)) |
| 1692 | goto exit_unlock; | 1692 | goto exit_unlock; |
| 1693 | 1693 | ||
| 1694 | err = 0; | ||
| 1694 | reply = ovs_vport_cmd_build_info(vport, info->snd_portid, info->snd_seq, | 1695 | reply = ovs_vport_cmd_build_info(vport, info->snd_portid, info->snd_seq, |
| 1695 | OVS_VPORT_CMD_NEW); | 1696 | OVS_VPORT_CMD_NEW); |
| 1696 | if (IS_ERR(reply)) { | 1697 | if (IS_ERR(reply)) { |