| From 46eaeb66ceec2c54898915c8717c0a28a5de3bd7 Mon Sep 17 00:00:00 2001 |
| From: Jeffrey Mitchell <jeffrey.mitchell@starlab.io> |
| Date: Wed, 5 Aug 2020 12:23:19 -0500 |
| Subject: [PATCH] nfs: Fix getxattr kernel panic and memory overflow |
| |
| commit b4487b93545214a9db8cbf32e86411677b0cca21 upstream. |
| |
| Move the buffer size check to decode_attr_security_label() before memcpy() |
| Only call memcpy() if the buffer is large enough |
| |
| Fixes: aa9c2669626c ("NFS: Client implementation of Labeled-NFS") |
| Signed-off-by: Jeffrey Mitchell <jeffrey.mitchell@starlab.io> |
| [Trond: clean up duplicate test of label->len != 0] |
| Signed-off-by: Trond Myklebust <trond.myklebust@hammerspace.com> |
| Signed-off-by: Paul Gortmaker <paul.gortmaker@windriver.com> |
| |
| diff --git a/fs/nfs/nfs4proc.c b/fs/nfs/nfs4proc.c |
| index 8db0b66086bd..f3c17c8fb022 100644 |
| --- a/fs/nfs/nfs4proc.c |
| +++ b/fs/nfs/nfs4proc.c |
| @@ -5729,8 +5729,6 @@ static int _nfs4_get_security_label(struct inode *inode, void *buf, |
| return ret; |
| if (!(fattr.valid & NFS_ATTR_FATTR_V4_SECURITY_LABEL)) |
| return -ENOENT; |
| - if (buflen < label.len) |
| - return -ERANGE; |
| return 0; |
| } |
| |
| diff --git a/fs/nfs/nfs4xdr.c b/fs/nfs/nfs4xdr.c |
| index 6a92a6eac911..d554baabef3e 100644 |
| --- a/fs/nfs/nfs4xdr.c |
| +++ b/fs/nfs/nfs4xdr.c |
| @@ -4163,7 +4163,11 @@ static int decode_attr_security_label(struct xdr_stream *xdr, uint32_t *bitmap, |
| return -EIO; |
| if (len < NFS4_MAXLABELLEN) { |
| if (label) { |
| - memcpy(label->label, p, len); |
| + if (label->len) { |
| + if (label->len < len) |
| + return -ERANGE; |
| + memcpy(label->label, p, len); |
| + } |
| label->len = len; |
| label->pi = pi; |
| label->lfs = lfs; |
| -- |
| 2.27.0 |
| |