CIFS: Rename bug on servers not supporting inode numbers
aia21 at cam.ac.uk
Thu Nov 3 09:20:17 MDT 2011
Our CIFS server problems seem to have no end… The Novell CIFS server does not support server inode numbers (when I try the mount option I get the message it is being turned off as server does not support it) and thus each inode gets a different number each time it is accessed and it gets a different number again for each readdir call.
The fun happens with rename() when the rename source and target only differ in case, e.g.
mv foo Foo
The result? Because of the difference in inode numbers, the request gets through to the CIFS module which promptly does:
rc = cifs_do_rename(…)
And because the cifs_unlink() just removed the source of the rename (as it is the same as the target), "rc" comes back as -ENOENT.
And indeed the file is gone so we just lost the user's file for ever. )-:
We are tossing around ideas how to fix this but we would be interested in your input as to what you think the fix should be.
In any case this probably should be fixed in the standard kernel CIFS module, too, and not just for us locally as this presumably affects anyone who is using the CIFS module against case-insensitive, non-server-inode-number-supporting CIFS servers...
Anton Altaparmakov <aia21 at cam.ac.uk> (replace at with @)
Unix Support, Computing Service, University of Cambridge, CB2 3QH, UK
Linux NTFS maintainer, http://www.linux-ntfs.org/
More information about the samba-technical