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

Re: [Xen-devel] [PATCH 6/6] tools/libx{l, c}: Fix trivial Coverity defects in migration v2 code

On Mon, 2015-07-20 at 11:37 +0100, Andrew Cooper wrote:
> All of these are UNUSED_VALUE defects where a default vaule is


> unconditionally overwritten.  They are not particularly interesting,
> bug wise, but keeping these defects at bay helps prevent real bugs
> going unnoticed in the volume.

The flip side is that if the code uses the "init everything and goto
out on error" idiom then the init might need to be put back if the code
changes, hopefully even a regular compiler would spot this though.

> No functional change.
> Signed-off-by: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>

Acked-by: Ian Campbell <Ian.Campbell@xxxxxxxxxx>

Xen-devel mailing list



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