[haiku-commits] Re: haiku: hrev52091 - in src: add-ons/kernel/drivers/network/wlan/iprowifi2200/dev/iwi . libs/compat add-ons/kernel/busses/scsi/ahci

  • From: waddlesplash <waddlesplash@xxxxxxxxx>
  • To: haiku-commits@xxxxxxxxxxxxx
  • Date: Tue, 21 Aug 2018 20:57:01 -0400

On Tue, Aug 21, 2018 at 4:15 PM Axel Dörfler <axeld@xxxxxxxxxxxxxxxx> wrote:

If the FS mount call actually succeeds, you should call unmount to give
it the chance to free any resources it may have acquired.

IMO, if the FS mount call returns B_OK but leaves this field
uninitialized, it probably means the internal state of the filesystem
is very confused, and the VFS layer should probably not call unmount
for fear some other KDL.

Alternatively, one could add a panic there, as that really shouldn't
happen. In any case, it would be nice to log a message about the
misbehavior, similar to what happens in line 7547 [1].

Will do.

-waddlesplash

Other related posts: