Bug 7223 - implementation of rename beyond filesystem boundaries is too limited
Summary: implementation of rename beyond filesystem boundaries is too limited
Alias: None
Product: Samba 3.5
Classification: Unclassified
Component: File services (show other bugs)
Version: 3.5.0
Hardware: All All
: P3 normal
Target Milestone: ---
Assignee: Volker Lendecke
QA Contact: Samba QA Contact
Depends on:
Reported: 2010-03-08 11:25 UTC by Björn Jacke
Modified: 2011-06-17 07:51 UTC (History)
0 users

See Also:
metze: review-

patch for 3.5.0 (16.53 KB, patch)
2010-03-08 11:29 UTC, Björn Jacke
jra: review+

Note You need to log in before you can comment on or make changes to this bug.
Description Björn Jacke 2010-03-08 11:25:48 UTC
- on huge files clients can timeout
- we lose ACLs and EA information

threrfore we should by default return NT_STATUS_NOT_SAME_DEVICE.


address this problem. They make the old limited cross filesystem rename support an optional feature which can be enabled via the crossrename vfs module.
Comment 1 Björn Jacke 2010-03-08 11:29:10 UTC
Created attachment 5471 [details]
patch for 3.5.0
Comment 2 Stefan Metzmacher 2010-04-13 07:07:22 UTC
Comment on attachment 5471 [details]
patch for 3.5.0

I think the patch is fine, but I'm not sure we should change the behavior between 3.5.x release would cause trouble in existing setups which just blindly upgrade the package.
Comment 3 Stefan Metzmacher 2010-04-13 08:12:48 UTC
I think we should not backport the change
Comment 4 Björn Jacke 2010-04-13 08:32:57 UTC
for the records in case someone stumbles upon it: in 3.5.x we stay upstream with  correct cross-fs rename behaviour of subdirectories and at the same time broken EA+ACL lossy cross-fs renames of single files. This will only be fixed for the 3.6 series.
Comment 5 Jeremy Allison 2010-04-13 14:04:19 UTC
Comment on attachment 5471 [details]
patch for 3.5.0

+1 from me. I concur on making this 3.6.0 only.