Testing for e == 0 after exiting the function in any other case a couple of lines earlier is useless. Found-by: Coverity Scan Signed-off-by: Patrick Georgi <patrick@xxxxxxxxxxxxxx> --- src/cbootimage.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/cbootimage.c b/src/cbootimage.c index 5f746e3..62328b4 100644 --- a/src/cbootimage.c +++ b/src/cbootimage.c @@ -211,7 +211,7 @@ main(int argc, char *argv[]) if (enable_debug) { /* Debugging information... */ - printf("bct size: %d\n", e == 0 ? context.bct_size : -1); + printf("bct size: %d\n", context.bct_size); } /* Open the raw output file. */ -- 1.9.1 -- To unsubscribe from this list: send the line "unsubscribe linux-tegra" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html