From: Colin King <colin.king@xxxxxxxxxxxxx> Date: Thu, 11 May 2017 19:29:40 +0100 > From: Colin Ian King <colin.king@xxxxxxxxxxxxx> > > The error status err is initialized as zero and then being checked > several times to see if it is less than zero even when it has not > been updated. It may seem that the err should be assigned to the > return code of the call to the various *offload_en_set calls and > then we check for failure, however, these functions are void and > never actually return any status. > > Since these error checks are redundant we can remove these > as well as err and the error exit label err_exit. > > Detected by CoverityScan, CID#1398313 and CID#1398306 ("Logically > dead code") > > Signed-off-by: Colin Ian King <colin.king@xxxxxxxxxxxxx> Applied, thank you. -- To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html