[Intrepid, Jaunty] SRU cifs+ipv6 regression

Stefan Bader stefan.bader at canonical.com
Thu Jan 22 16:08:03 GMT 2009


Tim Gardner wrote:
> Stefan Bader wrote:
>> 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.
>>
> 
> ACK - Please mark as a SAUCE patch since this one will surely originate
> from upstream in a stable update. You should apply it to Jaunty as well.
> 
Applied to Intrepid and Jaunty

-- 

When all other means of communication fail, try words!





More information about the kernel-team mailing list