FAQ Search Today's Posts Mark Forums Read
» Video Reviews

» Linux Archive

Linux-archive is a website aiming to archive linux email lists and to make them easily accessible for linux users/developers.


» Sponsor

» Partners

» Sponsor

Go Back   Linux Archive > Ubuntu > Ubuntu Kernel Team

 
 
LinkBack Thread Tools
 
Old 01-22-2009, 02:36 PM
Stefan Bader
 
Default SRU cifs+ipv6 regression

This is a multi-part message in MIME format.
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@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!
 
Old 01-22-2009, 02:46 PM
Tim Gardner
 
Default SRU cifs+ipv6 regression

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@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.

--
Tim Gardner tim.gardner@canonical.com

--
kernel-team mailing list
kernel-team@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/kernel-team
 
Old 01-22-2009, 03:08 PM
Stefan Bader
 
Default SRU cifs+ipv6 regression

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@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!



--
kernel-team mailing list
kernel-team@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/kernel-team
 

Thread Tools




All times are GMT. The time now is 09:21 PM.

VBulletin, Copyright ©2000 - 2014, Jelsoft Enterprises Ltd.
Content Relevant URLs by vBSEO ©2007, Crawlability, Inc.
Copyright 2007 - 2008, www.linux-archive.org