[Intrepid, Jaunty] SRU cifs+ipv6 regression

Stefan Bader stefan.bader at canonical.com
Thu Jan 22 15:36:43 UTC 2009


https://bugs.launchpad.net/ubuntu/+source/linux/+bug/318565

SRU justification:

Impact: The latest stable updates[1] to Intrepid introduced a regression in 
cifs when used with ipv6 addresses. The socket address was stored in a 
container on the stack but this was not big enough for ipv6 addresses. Failure 
was only observed on 32bit system. Code is from upstream, so Jaunty currently 
is affected, too.

Fix: The following patch by Jeff Layton replaces sockaddr by sockaddr_storage 
which is large enough to contain ipv4 and ipv6 socket structures.

Test: Without this patch on a 32bit system the following command will result in 
an oops: mount -t cifs -o user=test,ip=2001:a60:f001:1::69 //fileserv/pub /pub
After applying the patch this works ok. Patch will go upstream soon.

[1] commit e1a30a33db03c2a87f49d0c164c5ade0a0a6eb97
     Author: Steve French <sfrench at us.ibm.com>
     Date:   Fri Nov 21 14:23:07 2008 +0530
         cifs: clean up server protocol handling
         commit 3ec332ef7a38c2327e18d087d4120a8e3bd3dc6e upstream.

-- 

When all other means of communication fail, try words!


-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-cifs-make-sure-we-allocate-enough-storage-for-socke.patch
Type: text/x-diff
Size: 3539 bytes
Desc: not available
URL: <https://lists.ubuntu.com/archives/kernel-team/attachments/20090122/3174428e/attachment.patch>


More information about the kernel-team mailing list