Re: kern/52993 (case sensitive HFS+ can return data from wrong file)

[Harold Gutch <logix%foobar.franken.de@localhost>]

The following reply was made to PR kern/52993; it has been noted by GNATS.

From: Harold Gutch <logix%foobar.franken.de@localhost>
To: gnats-bugs%NetBSD.org@localhost
Cc: 
Subject: Re: kern/52993 (case sensitive HFS+ can return data from wrong file)
Date: Sun, 30 Dec 2018 17:08:19 +0100

 On Fri, Dec 28, 2018 at 04:15:00AM +0000, Sevan Janiyan wrote:
 >  With the supplied patch the issue is still not resolved.
 
 It works for me.  I have been using it all this time on a Raspberry Pi 1
 (albeit only under light load) with multiple case sensitive HFS+ file
 systems.
 
 This here is from a vanilla 8.0 amd64 installation in VirtualBox,
 with your disk image:
 
 localhost# mv /stand/amd64/8.0/modules/hfs/hfs.kmod /stand/amd64/8.0/modules/hfs/hfs_orig.kmod
 localhost# mv /stand/amd64/8.0/modules/hfs /stand/amd64/8.0/modules/hfs_orig
 localhost# cd /usr/src/sys/fs/hfs
 localhost# patch -p4 < patch
 localhost# cd /usr/src
 localhost# ./build.sh modules
 localhost# mv /usr/obj/destdir.amd64/stand/amd64/8.0/modules/hfs /stand/amd64/8.0/modules/
 
 localhost# vndconfig vnd0 kern52993-test.dmg
 localhost# modload hfs_orig
 localhost# mount -t hfs -o ro /dev/dk0 /mnt
 localhost# ls -li /mnt/test /mnt/test123
 23 -rw-r--r--  1 501  staff  4 Feb  9  2018 /mnt/test
 23 -rw-r--r--  1 501  staff  4 Feb  9  2018 /mnt/test123
 localhost# cat /mnt/test /mnt/test123
 foo
 foo
 localhost# umount /mnt
 localhost# modunload hfs
   [ although the module's file name/directory is "hfs_orig",
     the module's (internal) name still is "hfs" ]
 
 localhost# modload hfs
 localhost# mount -t hfs -o ro /dev/dk0 /mnt
 localhost# ls -li /mnt/test /mnt/test123
 23 -rw-r--r--  1 501  staff  4 Feb  9  2018 /mnt/test
 24 -rw-r--r--  1 501  staff  4 Feb  9  2018 /mnt/test123
 localhost# cat /mnt/test /mnt/test123
 foo
 bar
 localhost#
 
 If you want I can redo this test with current, but as sys/fs/hfs is
 virtually unchanged this shouldn't change anything.  I am not sure
 what else I can add right now.
 
 Or do you have some other issue with the patched module?
 
 
   Harold