On Fri, Aug 05, 2011 at 10:01:47AM -0500, Steve French wrote: > On Fri, Aug 5, 2011 at 9:54 AM, Al Viro <viro@xxxxxxxxxxxxxxxxxx> wrote: > > On Fri, Aug 05, 2011 at 09:02:40AM -0400, Jeff Layton wrote: > >> The loop around lookup_one_len doesn't handle the case where it might > >> return a negative dentry, which can cause an oops on the next pass > >> through the loop. Check for that and break out of the loop with an > >> error of -ENOENT if there is one. > > > > Ouch. ?Nice catch and that one is my damn fault. ?Do you want that one > > to go through cifs or vfs tree? ?ACK, in any case... > > Probably safer to go through my tree, reduce chance of merge conflict > Was planning on sending a merge request up soon in any case. OK Acked-by: Al Viro <viro@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