debdiff with the proposed fix ** Description changed:
- Create a file with the following contents, making sure that the samba - location is available and is mountable by guests and that the mountpoint - also exists and is a directory: + SRU justification : + + During a remount of a cifs filesystem, the mtab file is not properly + updated, which leads to a doubled entry of the same filesystem in the + /etc/mtab file. + + Impact : + + df and other tools incorrectly report the CIFS file system as mounted + twice + + Fix : + + This has been fixed upstream in version 5.3 which is present in Quantal + and more recent releases. The following upstream commit provides the fix + : + + https://git.samba.org/?p=cifs- + utils.git;a=commitdiff_plain;h=f46dd7661cfb87257c95081fc2071c934bfbbb16 + + Test Case : + + - An accessible SAMBA share is required on host 'sambahost' + - the user=,password= options are only required to avoid password prompt + + $ mkdir /mnt/target + $ mount -o user=,password= //sambahost/test /mnt/target + $ mount -o remount /mnt/target + $ df | grep target + //sambahost/test 7033648 1278124 5402656 20% /mnt/target + //sambahost/test 7033648 1278124 5402656 20% /mnt/target + + With the proposed fix, there would be only one /mnt/target entry + + Regression : + + This fix is already present in Q & R. It only introduce a del_mtab + function that is only called when MS_REMOUNT is set on the file system. + + + Original description of the problem : + + + Create a file with the following contents, making sure that the samba location is available and is mountable by guests and that the mountpoint also exists and is a directory: # cat /etc/puppet/manifests/tst.pp package{'samba-client': } mount{'home directory mount': - name => '/mnt/target', - atboot => 'true', - device => '//localhost/test', - ensure => 'mounted', - fstype => 'cifs', - require => [Package['samba-client']] + name => '/mnt/target', + atboot => 'true', + device => '//localhost/test', + ensure => 'mounted', + fstype => 'cifs', + require => [Package['samba-client']] } 4.2- Apply the puppet config: # puppet apply /etc/puppet/manifests/tst.pp 4.3- Verify that the mounted FS shows twice: root@ubuntu:~# mount -t cifs //localhost/test on /mnt/target type cifs (rw) //localhost/test on /mnt/target type cifs (rw) ** Summary changed: - puppet double mounts CIFS mounts + mount.cifs double mounts CIFS mounts ** Summary changed: - mount.cifs double mounts CIFS mounts + mount.cifs incorrectly update /etc/mtab when -o remount is used ** Changed in: cifs-utils (Ubuntu Precise) Status: Confirmed => In Progress ** Patch added: "cifs-utils-lp1144612.debdiff" https://bugs.launchpad.net/ubuntu/+source/cifs-utils/+bug/1144612/+attachment/3654742/+files/cifs-utils-lp1144612.debdiff -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1144612 Title: mount.cifs incorrectly update /etc/mtab when -o remount is used To manage notifications about this bug go to: https://bugs.launchpad.net/ubuntu/+source/cifs-utils/+bug/1144612/+subscriptions -- ubuntu-bugs mailing list [email protected] https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
