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

  • From: Axel Dörfler <axeld@xxxxxxxxxxxxxxxx>
  • To: haiku-commits@xxxxxxxxxxxxx
  • Date: Tue, 10 Jul 2018 09:17:49 +0200

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.

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.

Bye,
   Axel.

Other related posts: