On 2010-10-18, at 04:05, Jan Kara wrote: > On Sat 16-10-10 19:36:59, Theodore Ts'o wrote: >> @@ -871,6 +872,16 @@ static struct buffer_head *ext3_find_entry(struct inode *dir, >> namelen = entry->len; >> if (namelen > EXT3_NAME_LEN) >> return NULL; >> + if ((namelen < 2) && (name[0] == '.') && >> + (name[1] == '.' || name[1] == '0')) { > > This condition looks wrong... I suspect it should rather be: > (namelen <= 2) && (name[0] == '.') && (name[1] == '.' || name[1] == 0) > ^^^ change here and here ^^^ I think it is preferable to use '\0' for the trailing NUL. Cheers, Andreas -- To unsubscribe from this list: send the line "unsubscribe linux-ext4" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html