OGAWA Hirofumi <hirofumi@xxxxxxxxxxxxxxxxxx> writes: >> + if (parent && (len < FAT_FID_SIZE_WITH_PARENT)) { >> + *lenp = FAT_FID_SIZE_WITH_PARENT; >> + return 255; >> + } else if (len < FAT_FID_SIZE_WITHOUT_PARENT) { >> + *lenp = FAT_FID_SIZE_WITHOUT_PARENT; >> + return 255; We would be better to use FILEID_INVALID on all places, instead of 255. -- OGAWA Hirofumi <hirofumi@xxxxxxxxxxxxxxxxxx> -- 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