From patchwork Wed Apr 24 23:04:28 2013 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: "J. Bruce Fields" X-Patchwork-Id: 2486811 Return-Path: X-Original-To: patchwork-linux-nfs@patchwork.kernel.org Delivered-To: patchwork-process-083081@patchwork2.kernel.org Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by patchwork2.kernel.org (Postfix) with ESMTP id CB8D4DF25A for ; Wed, 24 Apr 2013 23:04:35 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932178Ab3DXXEc (ORCPT ); Wed, 24 Apr 2013 19:04:32 -0400 Received: from fieldses.org ([174.143.236.118]:47523 "EHLO fieldses.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932071Ab3DXXEb (ORCPT ); Wed, 24 Apr 2013 19:04:31 -0400 Received: from bfields by fieldses.org with local (Exim 4.76) (envelope-from ) id 1UV8jk-0006dC-Tk; Wed, 24 Apr 2013 19:04:28 -0400 Date: Wed, 24 Apr 2013 19:04:28 -0400 From: "J. Bruce Fields" To: Steve Dickson Cc: Trond Myklebust , "J. Bruce Fields" , "David P. Quigley" , Linux NFS list , Linux FS devel list , Linux Security List , SELinux List Subject: Re: [PATCH 16/17] NFSD: Server implementation of MAC Labeling Message-ID: <20130424230428.GT20275@fieldses.org> References: <1366834683-29075-1-git-send-email-SteveD@redhat.com> <1366834683-29075-17-git-send-email-SteveD@redhat.com> <20130424212827.GK20275@fieldses.org> <51785962.7080008@RedHat.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <51785962.7080008@RedHat.com> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-nfs-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-nfs@vger.kernel.org On Wed, Apr 24, 2013 at 06:14:58PM -0400, Steve Dickson wrote: > On 24/04/13 17:28, J. Bruce Fields wrote: > >> @@ -3231,11 +3340,13 @@ nfsd4_encode_setattr(struct nfsd4_compoundres *resp, __be32 nfserr, struct nfsd4 > >> > WRITE32(2); > >> > WRITE32(0); > >> > WRITE32(0); > >> > + WRITE32(0); > > Note that "2" above is the length of the bitmap field that follows. > > > >> > } > >> > else { > >> > WRITE32(2); > >> > WRITE32(setattr->sa_bmval[0]); > >> > WRITE32(setattr->sa_bmval[1]); > >> > + WRITE32(setattr->sa_bmval[2]); > > Ditto. > Right... I did miss that.... With that change the pynfs tests all pass this time. --b. --- To unsubscribe from this list: send the line "unsubscribe linux-nfs" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html diff --git a/fs/nfsd/nfs4xdr.c b/fs/nfsd/nfs4xdr.c index 75b5b6d..7bbc698 100644 --- a/fs/nfsd/nfs4xdr.c +++ b/fs/nfsd/nfs4xdr.c @@ -3310,13 +3310,13 @@ nfsd4_encode_setattr(struct nfsd4_compoundres *resp, __be32 nfserr, struct nfsd4 RESERVE_SPACE(12); if (nfserr) { - WRITE32(2); + WRITE32(3); WRITE32(0); WRITE32(0); WRITE32(0); } else { - WRITE32(2); + WRITE32(3); WRITE32(setattr->sa_bmval[0]); WRITE32(setattr->sa_bmval[1]); WRITE32(setattr->sa_bmval[2]);