On Sat, May 06, 2017 at 11:01:05PM +0100, Colin King wrote: > From: Colin Ian King <colin.king@xxxxxxxxxxxxx> > > Variable ret is assigned to zero and is always zero throughout the > function. Thus the check for ret being less than zero is always > false and so mapping_set_error always has an -EIO error passed to > it. Hence we can remove the redundant assignment and check on ret. > > Detected by CoverityScan, CID#1414312 ("Logically dead code") > > Signed-off-by: Colin Ian King <colin.king@xxxxxxxxxxxxx> Reviewed-by: David Sterba <dsterba@xxxxxxxx> -- 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