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

[Ian Campbell <ian.campbell@xxxxxxxxxx>]

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

"value".

> 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
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel