mirror of
https://github.com/raspberrypi/linux.git
synced 2025-12-06 18:09:56 +00:00
NFSv4.2: Fix a memory stomp in decode_attr_security_label
[ Upstream commit43c1031f71] We must not change the value of label->len if it is zero, since that indicates we stored a label. Fixes:b4487b9354("nfs: Fix getxattr kernel panic and memory overflow") Signed-off-by: Trond Myklebust <trond.myklebust@hammerspace.com> Signed-off-by: Sasha Levin <sashal@kernel.org>
This commit is contained in:
committed by
Greg Kroah-Hartman
parent
739c5c8ec7
commit
01aabe3cff
@@ -4236,12 +4236,10 @@ static int decode_attr_security_label(struct xdr_stream *xdr, uint32_t *bitmap,
|
|||||||
return -EIO;
|
return -EIO;
|
||||||
bitmap[2] &= ~FATTR4_WORD2_SECURITY_LABEL;
|
bitmap[2] &= ~FATTR4_WORD2_SECURITY_LABEL;
|
||||||
if (len < NFS4_MAXLABELLEN) {
|
if (len < NFS4_MAXLABELLEN) {
|
||||||
if (label) {
|
if (label && label->len) {
|
||||||
if (label->len) {
|
|
||||||
if (label->len < len)
|
if (label->len < len)
|
||||||
return -ERANGE;
|
return -ERANGE;
|
||||||
memcpy(label->label, p, len);
|
memcpy(label->label, p, len);
|
||||||
}
|
|
||||||
label->len = len;
|
label->len = len;
|
||||||
label->pi = pi;
|
label->pi = pi;
|
||||||
label->lfs = lfs;
|
label->lfs = lfs;
|
||||||
|
|||||||
Reference in New Issue
Block a user