[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH 2/3] xen-block: remove redundant assignment

To: Paul Durrant <paul.durrant@xxxxxxxxxx>
From: Anthony PERARD <anthony.perard@xxxxxxxxxx>
Date: Mon, 18 Feb 2019 14:43:57 +0000
Cc: Kevin Wolf <kwolf@xxxxxxxxxx>, Peter Maydell <peter.maydell@xxxxxxxxxx>, Stefano Stabellini <sstabellini@xxxxxxxxxx>, qemu-block@xxxxxxxxxx, qemu-devel@xxxxxxxxxx, Max Reitz <mreitz@xxxxxxxxxx>, xen-devel@xxxxxxxxxxxxxxxxxxxx
Delivery-date: Mon, 18 Feb 2019 14:44:11 +0000
List-id: Xen developer discussion <xen-devel.lists.xenproject.org>

On Fri, Feb 15, 2019 at 04:25:32PM +0000, Paul Durrant wrote:
> The assignment to 'p' is unnecessary as the code will either goto 'invalid'
> or p will get overwritten.
> 
> Spotted by Coverity: CID 1398638
> 
> Reported-by: Peter Maydell <peter.maydell@xxxxxxxxxx>
> Signed-off-by: Paul Durrant <paul.durrant@xxxxxxxxxx>

Acked-by: Anthony PERARD <anthony.perard@xxxxxxxxxx>

-- 
Anthony PERARD

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/xen-devel

References:
- [Xen-devel] [PATCH 0/3] Coverity fixes
  - From: Paul Durrant
- [Xen-devel] [PATCH 2/3] xen-block: remove redundant assignment
  - From: Paul Durrant

Prev by Date: [Xen-devel] [qemu-mainline test] 133284: tolerable FAIL - PUSHED
Next by Date: Re: [Xen-devel] [PATCH SpectreV1+L1TF v6 3/9] x86/hvm: block speculative out-of-bound accesses
Previous by thread: [Xen-devel] [PATCH 2/3] xen-block: remove redundant assignment
Next by thread: [Xen-devel] [PATCH 3/3] xen-block: report error condition from vbd_name_to_disk()
Index(es):
- Date
- Thread

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.