On Dec 21, 2015, at 11:42 AM, Piotr Caban piotr@codeweavers.com wrote:
@@ -2214,11 +2205,7 @@ static int read_directory_getattrlist( int fd, IO_STATUS_BLOCK *io, void *buffer } else io->u.Status = restart_scan ? STATUS_NO_SUCH_FILE : STATUS_NO_MORE_FILES; }
else if ((errno == ENOENT || errno == ENOTDIR) && !get_dir_case_sensitivity("."))
{
io->u.Status = restart_scan ? STATUS_NO_SUCH_FILE : STATUS_NO_MORE_FILES;
ret = 0;
}
else ret = -1;
Setting ret to -1 here is redundant. It will already be -1 if getattrlist() failed. Other than that, looks good to me.
} else ret = -1;
-Ken