diff options
| author |   Yunjian Wang <wangyunjian@huawei.com> | 2017-12-04 14:02:19 +0800 |
|---|---|---|
| committer |   Michael S. Tsirkin <mst@redhat.com> | 2017-12-07 18:34:52 +0200 |
| commit | 03e9f8a05bce7330bcd9c5cc54c8e42d0fcbf993 (patch) | |
| tree | 402ab23f3edf73fcea8a110b7176bb9c76e57db1 /drivers/net/virtio_net.c | |
| parent | virtio_mmio: add cleanup for virtio_mmio_remove (diff) | |
| download | linux-dev-03e9f8a05bce7330bcd9c5cc54c8e42d0fcbf993.tar.xz linux-dev-03e9f8a05bce7330bcd9c5cc54c8e42d0fcbf993.zip | |
virtio_net: fix return value check in receive_mergeable()
The function virtqueue_get_buf_ctx() could return NULL, the return
value 'buf' need to be checked with NULL, not value 'ctx'.
Signed-off-by: Yunjian Wang <wangyunjian@huawei.com>
Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
Diffstat (limited to 'drivers/net/virtio_net.c')
| -rw-r--r-- | drivers/net/virtio_net.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/net/virtio_net.c b/drivers/net/virtio_net.c index 511f8339fa96..7927e28f5336 100644 --- a/drivers/net/virtio_net.c +++ b/drivers/net/virtio_net.c @@ -714,7 +714,7 @@ static struct sk_buff *receive_mergeable(struct net_device *dev, int num_skb_frags; buf = virtqueue_get_buf_ctx(rq->vq, &len, &ctx); - if (unlikely(!ctx)) { + if (unlikely(!buf)) { pr_debug("%s: rx error: %d buffers out of %d missing\n", dev->name, num_buf, virtio16_to_cpu(vi->vdev, |