Attachment 633405 Details for Bug 869942 – Patch for fs/nfs/nfs4proc.c which fixes the issue here

[patch] Patch for fs/nfs/nfs4proc.c which fixes the issue here

nfs4proc.c.patch (text/plain), 2.11 KB, created by Albert Flügel on 2012-10-25 15:16:21 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Albert Flügel

Created: 2012-10-25 15:16:21 UTC

Size: 2.11 KB

patch

obsolete

>--- nfs4proc.c	2012-08-31 14:05:18.000000000 +0200
>+++ /home/tdscaf/nfs4proc.c	2012-10-25 17:11:44.000000000 +0200
>@@ -3411,16 +3411,17 @@
> 	return ret;
> }
> 
>-static void nfs4_write_cached_acl(struct inode *inode, const char *buf, size_t acl_len)
>+static void nfs4_write_cached_acl(struct inode *inode, struct page **pages, size_t pgbase, size_t acl_len)
> {
> 	struct nfs4_cached_acl *acl;
>+	size_t buflen = sizeof(*acl) + acl_len;
> 
>-	if (buf && acl_len <= PAGE_SIZE) {
>-		acl = kmalloc(sizeof(*acl) + acl_len, GFP_KERNEL);
>+	if (buflen <= PAGE_SIZE) {
>+		acl = kmalloc(buflen, GFP_KERNEL);
> 		if (acl == NULL)
> 			goto out;
> 		acl->cached = 1;
>-		memcpy(acl->data, buf, acl_len);
>+		_copy_from_pages(acl->data, pages, pgbase, acl_len);
> 	} else {
> 		acl = kmalloc(sizeof(*acl), GFP_KERNEL);
> 		if (acl == NULL)
>@@ -3453,13 +3454,12 @@
> 	struct nfs_getaclres res = {
> 		.acl_len = buflen,
> 	};
>-	void *resp_buf;
> 	struct rpc_message msg = {
> 		.rpc_proc = &nfs4_procedures[NFSPROC4_CLNT_GETACL],
> 		.rpc_argp = &args,
> 		.rpc_resp = &res,
> 	};
>-	int ret = -ENOMEM, npages, i, acl_len = 0;
>+	int ret = -ENOMEM, npages, i, acl_len;
> 
> 	npages = (buflen + PAGE_SIZE - 1) >> PAGE_SHIFT;
> 	/* As long as we're doing a round trip to the server anyway,
>@@ -3484,7 +3484,6 @@
> 	 * the page we send as a guess */
> 	if (buf == NULL)
> 		res.acl_flags |= NFS4_ACL_LEN_REQUEST;
>-	resp_buf = page_address(pages[0]);
> 
> 	dprintk("%s  buf %p buflen %zu npages %d args.acl_len %zu\n",
> 		__func__, buf, buflen, npages, args.acl_len);
>@@ -3492,20 +3491,17 @@
> 	if (ret)
> 		goto out_free;
> 
>-	acl_len = res.acl_len - res.acl_data_offset;
>-	if (acl_len > args.acl_len)
>-		nfs4_write_cached_acl(inode, NULL, acl_len);
>-	else
>-		nfs4_write_cached_acl(inode, resp_buf + res.acl_data_offset,
>-				      acl_len);
>+	nfs4_write_cached_acl(inode, pages, res.acl_data_offset,
>+				      res.acl_len);
> 	if (buf) {
> 		ret = -ERANGE;
>+		acl_len = res.acl_len - res.acl_data_offset;
> 		if (acl_len > buflen)
> 			goto out_free;
> 		_copy_from_pages(buf, pages, res.acl_data_offset,
> 				acl_len);
> 	}
>-	ret = acl_len;
>+	ret = res.acl_len;
> out_free:
> 	for (i = 0; i < npages; i++)
> 		if (pages[i])

Actions: View | Diff

Attachments on bug 869942: 633216 | 633217 | 633405 | 634959 | 680079