[haiku-commits] Re: haiku: hrev52087 - src/system/kernel/fs

  • From: waddlesplash <waddlesplash@xxxxxxxxx>
  • To: haiku-commits@xxxxxxxxxxxxx
  • Date: Tue, 10 Jul 2018 09:48:17 -0400

On Tue, Jul 10, 2018, 3:18 AM Axel Dörfler <axeld@xxxxxxxxxxxxxxxx> wrote:

Am 10/07/2018 um 05:17 schrieb waddlesplash:
On Mon, Jul 9, 2018, 10:58 PM Rene Gollent <anevilyak@xxxxxxxxx
<mailto:anevilyak@xxxxxxxxx>> wrote:
   structure itself must be there. This shouldn't ever be NULL after a
   successful mount, otherwise there are deeper problems that will
simply
   be hit later.
That may very well be the case; but it's also true that users might use
slightly buggy filesystem drivers, and the kernel should tolerate that
fault here, no?

No, it should test this early, and not use that whole file system at all
in this case.

Interestingly, I ran into the same issue yesterday, also with an NTFS
partition -- must be some recent change.


I note that I'm on a hrev that predates korli's recent NTFS changes, so
that's unrelated at least.

I think the proper place for the check would be line 7526 -- the status
check after calling mount(). Does that make sense?

At any rate, this seems related to the automount changes, because as I
noted in my commit message deleting my mount server settings file caused
it to go away (there was an NTFS partition that had wound up in there.)
I can still mount the NTFS partition on its own, so if there is a
problem, it doesn't seem to be with this code then...

Definitely something someone should look into a bit deeper.


Well, I'm still new to the whole kernel game, so it probably won't be me...

-waddlesplash

Other related posts:

  1. Home
  2. haiku-commits
  3. Archive
  4. 07-2018