diff options
| author | Dan Carpenter <dan.carpenter@oracle.com> | 2012-05-13 12:44:18 +0400 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2012-05-13 23:47:34 +0400 |
| commit | 8aa51d64c1f526e43b1e7f89fb8b98c2fd583f4b (patch) | |
| tree | 8444efa46bbd8dbe4e01655aaa719d8d5257928e /net/openvswitch/datapath.c | |
| parent | b99215cdc6e191f5649687536d4fb0faa3d7f56e (diff) | |
| download | linux-8aa51d64c1f526e43b1e7f89fb8b98c2fd583f4b.tar.xz | |
openvswitch: checking wrong variable in queue_userspace_packet()
"skb" is non-NULL here, for example we dereference it in skb_clone().
The intent was to test "nskb" which was just set.
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Acked-by: Jesse Gross <jesse@nicira.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/openvswitch/datapath.c')
| -rw-r--r-- | net/openvswitch/datapath.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/net/openvswitch/datapath.c b/net/openvswitch/datapath.c index 777716bc80f7..e66341ec455c 100644 --- a/net/openvswitch/datapath.c +++ b/net/openvswitch/datapath.c @@ -321,7 +321,7 @@ static int queue_userspace_packet(int dp_ifindex, struct sk_buff *skb, return -ENOMEM; nskb = __vlan_put_tag(nskb, vlan_tx_tag_get(nskb)); - if (!skb) + if (!nskb) return -ENOMEM; nskb->vlan_tci = 0; |