Re: [Xen-devel] [PATCH] tools/Coverity: Audit of MISSING_BREAK defects

[Wei Liu <wei.liu2@xxxxxxxxxx>]

On Thu, Feb 12, 2015 at 08:08:33PM +0000, Andrew Cooper wrote:
> Coverity uses several heuristics to identify when one case statement
> legitimately falls through into the next, and a comment as the final item in a
> case statement is one heuristic (the assumption being that it is a
> justification for the fallthrough).
> 
> Use this to perform an audit of defects and hide the legitimate fallthroughs.
> 
> There are two bugfixes identified in the audit, both minor:
>  * 'n' command line handling for gtracestat
>  * BKSPC handling in xentop
> 
> All other identified defaults are legitimate fallthoughs
> 
> Signed-off-by: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
> Coverity-IDs: 1055464, 1055465, 1055467, 1055468, 1055481, 1055482
> CC: Ian Campbell <Ian.Campbell@xxxxxxxxxx>
> CC: Ian Jackson <Ian.Jackson@xxxxxxxxxxxxx>
> CC: Wei Liu <wei.liu2@xxxxxxxxxx>
> CC: Xen Coverity Team <coverity@xxxxxxx>

Acked-by: Wei Liu <wei.liu2@xxxxxxxxxx>

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel