<div dir="ltr"><br><br><div class="gmail_quote">On Thu, Sep 25, 2008 at 5:43 PM, Tero Roponen <span dir="ltr"><<a href="mailto:tero.roponen@gmail.com">tero.roponen@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
               if ((errno = -ext2_readdir(dirfile, &(struct fillstate){ .dirent = dirent }, tux3_filler)))<br>
                        return -errno;<br>
</blockquote><div>Shouldn't this check be done in ext2_readdir itself?? I though that was the main purpose of calling<br>ext2_readdir... to get any possible errors out of the way... <br></div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
+               if (!*dirent)<br>
+                       return -ENOENT;<br>
                tux3_getattr(dirent, &stat);<br>
                if (fuse_filler(buf, dirent, &stat, dirfile->f_pos)) {<br>
                        warn("fuse buffer full");<br>
</blockquote><div>Pranith. <br></div></div>
</div>