Skip to contentSkip to conversation

ecryptfs: Log function name only once in decode_and_decrypt_filename

PATCH RESEND In review Thorsten Blum 2 participants 2 messages Mar 10 – Mar 26 Ecrypt file system

lore
Share & tools
lore
#0 lore raw

PATCH RESEND 1/1 ecryptfs: Log function name only once in decode_and_decrypt_filename

ecryptfs_printk() already prints the function name using %s and __func__. Drop the redundant function name from the debug log message.

---
 fs/ecryptfs/crypto.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)
fs/ecryptfs/crypto.c +1 -2
index 3b59346d68c5..d4e714912429 100644
@@ -1925,8 +1925,7 @@ int ecryptfs_decode_and_decrypt_filename(char **plaintext_name,
 						  decoded_name_size);
 		if (rc) {
 			ecryptfs_printk(KERN_DEBUG,
-					"%s: Could not parse tag 70 packet from filename\n",
-					__func__);
+					"Could not parse tag 70 packet from filename\n");
 			goto out_free;
 		}
 	} else {
maintainer
#1 lore raw
Quoted 2 lines · Thorsten Blum

ecryptfs_printk() already prints the function name using %s and __func__. Drop the redundant function name from the debug log message.

Thank you! This has been applied to the next branch of the tyhicks/ecryptfs.git tree.

You can find a direct link below but please be aware that the commit hash is unstable and, therefore, the URL may not be valid in the future.

[1/1] ecryptfs: Log function name only once in decode_and_decrypt_filename

      https://git.kernel.org/tyhicks/ecryptfs/c/1601fe9e0423d813b1158a52e051bd3059f74197

Tyler