From: Damian Stewart <damian@frey.co.nz>
To: Brad Boyer <flar@allandria.com>
Cc: linux-fsdevel@vger.kernel.org
Subject: Re: PROBLEM: hfsplus filesystem allows opendir() on plain files
Date: Fri, 05 Dec 2008 09:53:51 +0100 [thread overview]
Message-ID: <4938EC1F.7060606@frey.co.nz> (raw)
In-Reply-To: <20081202222111.GA2738@cynthia.pants.nu>
Brad Boyer wrote:
> Maybe the check in the generic code to decide what is a "directory"
> should be changed? We could possibly check inode->i_fop->readdir.
Hi Brad,
Thanks for your reply.
This issue came up because at least one major project (the VLC video
player) is using 'opendir()' to determine whether or not a path is a file
or directory. (They probably ought to be using 'stat()', but they're not,
so this issue shows up.)
I've reported it on the VLC forums but only got very dismissive replies.
The few developers who posted bluntly said it was a 'kernel bug' so they
weren't willing to do anything about it; which is a shame, because it
leaves those VLC users with videos on HFS+ drives unable to play their
videos with the latest version of VLC.
Anyways, perhaps your suggestion is the right one, in this case.
Cheers
d
--
damian stewart | skype: damiansnz | damian@frey.co.nz
frey | live art with machines | http://www.frey.co.nz
next prev parent reply other threads:[~2008-12-05 8:54 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-12-02 15:26 PROBLEM: hfsplus filesystem allows opendir() on plain files Damian Stewart
2008-12-02 22:21 ` Brad Boyer
2008-12-05 8:53 ` Damian Stewart [this message]
2008-12-06 6:00 ` Brad Boyer
2008-12-05 9:39 ` Christoph Hellwig
2008-12-06 6:32 ` Brad Boyer
2008-12-06 6:47 ` Al Viro
2008-12-06 7:21 ` Brad Boyer
2008-12-06 22:31 ` Al Viro
2008-12-06 22:35 ` Al Viro
2008-12-06 12:06 ` Anton Altaparmakov
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=4938EC1F.7060606@frey.co.nz \
--to=damian@frey.co.nz \
--cc=flar@allandria.com \
--cc=linux-fsdevel@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).