linux-kernel - Re: [PATCH v2] ecryptfs: Drop redundant NUL terminations after calling ecryptfs_to

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <aUsBnAdoEmQW8wVb@yaupon>
Date: Tue, 23 Dec 2025 14:54:52 -0600
From: Tyler Hicks <code@...icks.com>
To: Thorsten Blum <thorsten.blum@...ux.dev>
Cc: Ard Biesheuvel <ardb@...nel.org>,
	Christian Brauner <brauner@...nel.org>,
	Eric Biggers <ebiggers@...nel.org>, ecryptfs@...r.kernel.org,
	linux-kernel@...r.kernel.org
Subject: Re: [PATCH v2] ecryptfs: Drop redundant NUL terminations after
 calling ecryptfs_to_hex

On 2025-11-20 23:17:57, Thorsten Blum wrote:
> ecryptfs_to_hex() already NUL-terminates the destination buffers. Drop
> the manual NUL terminations.
> 
> Signed-off-by: Thorsten Blum <thorsten.blum@...ux.dev>

Thanks again for the cleanup!

Acked-by: Tyler Hicks <code@...icks.com>

Tyler

> ---
> Changes in v2:
> - Remove all redundant NUL terminations after calling ecryptfs_to_hex
> - Link to v1: https://lore.kernel.org/lkml/20251120214535.5952-2-thorsten.blum@linux.dev/
> ---
>  fs/ecryptfs/debug.c    | 1 -
>  fs/ecryptfs/keystore.c | 3 ---
>  2 files changed, 4 deletions(-)
> 
> diff --git a/fs/ecryptfs/debug.c b/fs/ecryptfs/debug.c
> index cf6d0e8e25a1..c185a8cb5fe2 100644
> --- a/fs/ecryptfs/debug.c
> +++ b/fs/ecryptfs/debug.c
> @@ -28,7 +28,6 @@ void ecryptfs_dump_auth_tok(struct ecryptfs_auth_tok *auth_tok)
>  		ecryptfs_printk(KERN_DEBUG, " * passphrase type\n");
>  		ecryptfs_to_hex(salt, auth_tok->token.password.salt,
>  				ECRYPTFS_SALT_SIZE);
> -		salt[ECRYPTFS_SALT_SIZE * 2] = '\0';
>  		ecryptfs_printk(KERN_DEBUG, " * salt = [%s]\n", salt);
>  		if (auth_tok->token.password.flags &
>  		    ECRYPTFS_PERSISTENT_PASSWORD) {
> diff --git a/fs/ecryptfs/keystore.c b/fs/ecryptfs/keystore.c
> index 3a6283477d61..9f53069bbc7c 100644
> --- a/fs/ecryptfs/keystore.c
> +++ b/fs/ecryptfs/keystore.c
> @@ -908,7 +908,6 @@ ecryptfs_parse_tag_70_packet(char **filename, size_t *filename_size,
>  	(*packet_size) += s->packet_size_len;
>  	ecryptfs_to_hex(s->fnek_sig_hex, &data[(*packet_size)],
>  			ECRYPTFS_SIG_SIZE);
> -	s->fnek_sig_hex[ECRYPTFS_SIG_SIZE_HEX] = '\0';
>  	(*packet_size) += ECRYPTFS_SIG_SIZE;
>  	s->cipher_code = data[(*packet_size)++];
>  	rc = ecryptfs_cipher_code_to_string(s->cipher_string, s->cipher_code);
> @@ -1777,8 +1776,6 @@ int ecryptfs_parse_packet_set(struct ecryptfs_crypt_stat *crypt_stat,
>  			}
>  			ecryptfs_to_hex(new_auth_tok->token.password.signature,
>  					sig_tmp_space, tag_11_contents_size);
> -			new_auth_tok->token.password.signature[
> -				ECRYPTFS_PASSWORD_SIG_SIZE] = '\0';
>  			crypt_stat->flags |= ECRYPTFS_ENCRYPTED;
>  			break;
>  		case ECRYPTFS_TAG_1_PACKET_TYPE:
> -- 
> 2.51.1
>