Am 03.05.2015 um 20:15 schrieb Fabian Frederick: >> If the code can be reached you need to take a proper action of not the >> warning is bogus. > > As far as I understand, switch in affs_lookup() only updates inode number when > it's a link so we can simply add default:break to silence gcc warning or replace > by if(). Does the warning trigger with our default set of compiler warnings? Not all gcc warnings are useful. :) > affs_iget() switch on the other hand applies to any type of file. > Maybe we could add default: BUG() to avoid working on bad inode ? Or cancel the operation and return -EIO to userspace. Thanks, //richard -- To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html