NeilBrown 8e67c7
From: Olga Kornievskaia <kolga@netapp.com>
NeilBrown 8e67c7
Date: Mon, 29 Nov 2021 15:33:56 -0500
NeilBrown 8e67c7
Subject: [PATCH] NFSv4 only print the label when its queried
NeilBrown 8e67c7
Git-commit: 2c52c8376db7160a1dd8a681c61c9258405ef143
NeilBrown 8e67c7
Patch-mainline: v5.17
NeilBrown 8e67c7
References: git-fixes
NeilBrown 8e67c7
NeilBrown 8e67c7
When the bitmask of the attributes doesn't include the security label,
NeilBrown 8e67c7
don't bother printing it. Since the label might not be null terminated,
NeilBrown 8e67c7
adjust the printing format accordingly.
NeilBrown 8e67c7
NeilBrown 8e67c7
Signed-off-by: Olga Kornievskaia <kolga@netapp.com>
NeilBrown 8e67c7
Signed-off-by: Anna Schumaker <Anna.Schumaker@Netapp.com>
NeilBrown 8e67c7
Acked-by: NeilBrown <neilb@suse.com>
NeilBrown 8e67c7
NeilBrown 8e67c7
---
NeilBrown 8e67c7
 fs/nfs/nfs4xdr.c |    7 ++++---
NeilBrown 8e67c7
 1 file changed, 4 insertions(+), 3 deletions(-)
NeilBrown 8e67c7
NeilBrown 8e67c7
--- a/fs/nfs/nfs4xdr.c
NeilBrown 8e67c7
+++ b/fs/nfs/nfs4xdr.c
NeilBrown 8e67c7
@@ -4183,10 +4183,11 @@ static int decode_attr_security_label(st
NeilBrown 8e67c7
 		} else
NeilBrown 8e67c7
 			printk(KERN_WARNING "%s: label too long (%u)!\n",
NeilBrown 8e67c7
 					__func__, len);
NeilBrown 8e67c7
+		if (label && label->label)
NeilBrown 8e67c7
+			dprintk("%s: label=%.*s, len=%d, PI=%d, LFS=%d\n",
NeilBrown 8e67c7
+				__func__, label->len, (char *)label->label,
NeilBrown 8e67c7
+				label->len, label->pi, label->lfs);
NeilBrown 8e67c7
 	}
NeilBrown 8e67c7
-	if (label && label->label)
NeilBrown 8e67c7
-		dprintk("%s: label=%s, len=%d, PI=%d, LFS=%d\n", __func__,
NeilBrown 8e67c7
-			(char *)label->label, label->len, label->pi, label->lfs);
NeilBrown 8e67c7
 	return status;
NeilBrown 8e67c7
 
NeilBrown 8e67c7
 out_overflow: