[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] Re: [PATCH 2/6] virtio: net: remove dead assignment
From: |
Michael S. Tsirkin |
Subject: |
[Qemu-devel] Re: [PATCH 2/6] virtio: net: remove dead assignment |
Date: |
Wed, 13 Jan 2010 22:51:15 +0200 |
User-agent: |
Mutt/1.5.19 (2009-01-05) |
On Wed, Jan 13, 2010 at 04:24:43PM +0530, Amit Shah wrote:
> clang-analyzer points out value assigned to 'len' is not used.
>
> Signed-off-by: Amit Shah <address@hidden>
Acked-by: Michael S. Tsirkin <address@hidden>
> ---
> hw/virtio-net.c | 2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/hw/virtio-net.c b/hw/virtio-net.c
> index 02d9180..6e48997 100644
> --- a/hw/virtio-net.c
> +++ b/hw/virtio-net.c
> @@ -532,7 +532,7 @@ static ssize_t virtio_net_receive(VLANClientState *nc,
> const uint8_t *buf, size_
> int len, total;
> struct iovec sg[VIRTQUEUE_MAX_SIZE];
>
> - len = total = 0;
> + total = 0;
>
> if ((i != 0 && !n->mergeable_rx_bufs) ||
> virtqueue_pop(n->rx_vq, &elem) == 0) {
> --
> 1.6.2.5
>
>
- [Qemu-devel] [PATCH 0/6] Remove some dead assignments from clang analyzer report, Amit Shah, 2010/01/13
- [Qemu-devel] [PATCH 1/6] vl.c: Remove dead assignment, Amit Shah, 2010/01/13
- [Qemu-devel] [PATCH 2/6] virtio: net: remove dead assignment, Amit Shah, 2010/01/13
- [Qemu-devel] [PATCH 3/6] x86: translate.c: remove dead assignment, Amit Shah, 2010/01/13
- [Qemu-devel] [PATCH 4/6] hw/vga.c: remove dead assignment, Amit Shah, 2010/01/13
- [Qemu-devel] [PATCH 5/6] qcow2-refcount: remove dead assignment, Amit Shah, 2010/01/13
- [Qemu-devel] [PATCH 6/6] json-parser: remove dead increment, Amit Shah, 2010/01/13
- [Qemu-devel] Re: [PATCH 5/6] qcow2-refcount: remove dead assignment, Paolo Bonzini, 2010/01/13
- [Qemu-devel] Re: [PATCH 5/6] qcow2-refcount: remove dead assignment, Amit Shah, 2010/01/13
- [Qemu-devel] Re: [PATCH 2/6] virtio: net: remove dead assignment,
Michael S. Tsirkin <=
Re: [Qemu-devel] [PATCH 1/6] vl.c: Remove dead assignment, Anthony Liguori, 2010/01/13