Red Hat Bugzilla – Bug 431868
mounting CIFS subshare doesn't autoconvert prepath delimiters
Last modified: 2010-04-28 09:51:37 EDT
Looks like this has caused a regression when posix extensions are enabled. Even
though mount.cifs uses '/' as a prefixpath delimiter,
This bugzilla has Keywords: Regression.
Since no regressions are allowed between releases,
it is also being proposed as a blocker for this release.
Please resolve ASAP.
Note that this regression is actually caused by a userspace mount.cifs patch.
While the userspace patch fixes the situation where we're mounting a subshare of
a mount without posix extensions, it breaks this when posix extensions are enabled.
We have 2 choices -- pull the patch from samba for 5.2 and leave non-posix
mounts in a broken state, or try to fix this in kernel. I think fixing this in
kernel is probably the best solution, but if there are time constraints we can
go with the first option.
The fix shouldn't be terribly difficult. I'll hopefully have a patch later today.
Created attachment 294220 [details]
patch -- add routine to convert delimiters on the fly
This patch seems to fix it. There may be cleaner ways to do this using some of
the string handling routines built into the kernel though. I want to do a bit
more testing on this and make sure we're not breaking any other cases. Once I
do I'll plan to email Steve French on this and see if he can review it quickly
If so, it would be good to get this into a 5.2 respin (if there is one).
Patch sent to upstream list and Steve F. Awaiting his response...
This request was evaluated by Red Hat Product Management for inclusion in a Red
Hat Enterprise Linux maintenance release. Product Management has requested
further review of this request by Red Hat Engineering, for potential
inclusion in a Red Hat Enterprise Linux Update release for currently deployed
products. This request is not yet committed for inclusion in an Update
We've decided to pull the problem patch from Samba for 5.2 and focus on getting
this fixed the right way upstream. That probably won't happen until 5.3 though,
so resetting this for 5.3.
Development Management has reviewed and declined this request. You may appeal
this decision by reopening this request.
The upstream patch ended up looking quite a bit different than what was
originally proposed. I've also posted an updated userspace patch and an update
to the manpage to the upstream lists today.
You can download this test kernel from http://people.redhat.com/dzickus/el5
*** Bug 446433 has been marked as a duplicate of this bug. ***
An advisory has been issued which should help the problem
described in this bug report. This report is therefore being
closed with a resolution of ERRATA. For more information
on therefore solution and/or where to find the updated files,
please follow the link below. You may reopen this bug report
if the solution does not work for you.