On Tue, 2012-07-24 at 12:36 +0100, Sachin Prabhu wrote: > This fixes a bug introduced by commit > 5a00689930ab975fdd1b37b034475017e460cf2a > > Bruce Fields pointed out that the changes introduced by the patch will > cause the array npages to overflow if a size requested is >= > XATTR_SIZE_MAX. > > The patch also fixes the check for the length of the ACL returned. The > flag ACL_LEN_REQUEST has been renamed to NFS4_ACL_LEN_ONLY and apart > from indicating that the user is just interested in the ACL length when > making a request, it is also used to indicate if the xdr pages buffer > returned in response holds the complete ACL or just the length. > > Signed-off-by: Sachin Prabhu <sprabhu@xxxxxxxxxx> If this fix is going into stable, then the patch needs to be split up. The rename of ACL_LEN_REQUEST to NFS4_ACL_LEN_ONLY is a cleanup and should not be propagated to stable... Cheers Trond -- Trond Myklebust Linux NFS client maintainer NetApp Trond.Myklebust@xxxxxxxxxx www.netapp.com ��.n��������+%������w��{.n�����{��w���jg��������ݢj����G�������j:+v���w�m������w�������h�����٥