OGAWA Hirofumi <hirofumi@xxxxxxxxxxxxxxxxxx> writes: >>> Um... How do lseek() work? It sounds like to violate error code range. >> >> This is for read-write. As far as I know, >> - generic_file_llseek, >> - default_llseek >> - no_llseek >> >> doesn't call this function. > > It seems to allow to set negative value to ->f_pos, right? So, lseek() > returns (uses) it? BTW, another concern by negative "pos" value is, the following like code pos >> shift_bits it will break the above. So, I think it should be checked if not yet. -- 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