mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Jos Hulzink <josh@stack.nl>
To: OGAWA Hirofumi <hirofumi@mail.parknet.co.jp>
Cc: Linux Kernel Development <linux-kernel@vger.kernel.org>
Subject: VFS issues (was: Re: 2.5.5-pre1: mounting NTFS partitions -t VFAT)
Date: Tue, 19 Feb 2002 11:02:15 +0100 (CET)	[thread overview]
Message-ID: <20020219102539.J93925-100000@snail.stack.nl> (raw)
In-Reply-To: <874rken8ik.fsf@devron.myhome.or.jp>

On Mon, 18 Feb 2002, OGAWA Hirofumi wrote:

> Jos Hulzink <josh@stack.nl> writes:
>
> > What lacks is a fingerprint detector, and iirc -long time ago- FAT has a
> > very easy to detect fingerprint.
> >
> > I'll dig into FAT documentation tonight.
>
> I read the document repeatedly and did much tests. If you read the
> document, you may use BS_OEMName or BS_FilSysType, however, these
> don't have a meaning.

Hmmm. You seem to be right there. In my OS (IBM PC only) I checked the
partition table (see below).

The first question I want answered: Should I just call myself stupid for
trying to mount NTFS as VFAT, or should we consider this a real issue that
needs fixing ? (I see the problem as a generic problem. There must be
other combinations of filesystems and partition types that pass the
test, but are wrong). IMHO the latter, for every lost partition makes an
angry linux user.

Anyway. I have already been thinking further. Maybe I'm talking nonsense,
but I'll give it a try.

The type of a partition is written in the partition table, or something
similar. Maybe we should check that ?

While mounting a partition, the vfs layer tries to determine the partition
type, and passes that info to the filesystem driver, which checks whether
that partition type can be mounted by the driver. If no partition type is
provided by the vfs layer (for the partition type is not available in the
used partition table, or whatever), the fs driver must try to find out
itself.

If you don't want the calls to change, another option is to move the check
to the vfs. Problem there however is that the vfs needs a "filesystem <->
partition type" table. On the other hand, this way we can add extra
security without breaking anything.

Jos


  reply	other threads:[~2002-02-19 10:02 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2002-02-15 10:30 2.5.5-pre1: mounting NTFS partitions -t VFAT Jos Hulzink
2002-02-15 17:11 ` Anton Altaparmakov
2002-02-15 20:18 ` OGAWA Hirofumi
2002-02-18 12:55   ` Jos Hulzink
2002-02-18 14:56     ` OGAWA Hirofumi
2002-02-19 10:02       ` Jos Hulzink [this message]
2002-02-19 11:52         ` VFS issues (was: Re: 2.5.5-pre1: mounting NTFS partitions -t VFAT) Richard Russon
2002-02-19 12:48           ` Jos Hulzink
2002-02-19 13:34         ` Alexander Viro
2002-02-19 14:25         ` Denis Vlasenko
2002-02-19 11:03           ` Jos Hulzink
2002-03-09  9:50       ` [BUG] 2.5.6: IPv6 fails to initialize Jos Hulzink
2002-03-09 20:55         ` Ben Clifford
2002-03-11 17:58       ` David S. Miller
2002-02-19 12:21 VFS issues (was: Re: 2.5.5-pre1: mounting NTFS partitions -t VFAT) Andries.Brouwer

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=20020219102539.J93925-100000@snail.stack.nl \
    --to=josh@stack.nl \
    --cc=hirofumi@mail.parknet.co.jp \
    --cc=linux-kernel@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

all inboxes | Powered by JetHome®