public inbox for ecryptfs@vger.kernel.org
 help / color / mirror / Atom feed
* [PATCH 1/2] ecryptfs: Fix typo in ecryptfs_derive_iv function comment
@ 2026-03-30 10:35 Thorsten Blum
  2026-03-30 10:35 ` [PATCH 2/2] ecryptfs: Drop TODO comment in ecryptfs_derive_iv Thorsten Blum
  2026-04-02 22:32 ` [PATCH 1/2] ecryptfs: Fix typo in ecryptfs_derive_iv function comment Tyler Hicks
  0 siblings, 2 replies; 3+ messages in thread
From: Thorsten Blum @ 2026-03-30 10:35 UTC (permalink / raw)
  To: Tyler Hicks, Thorsten Blum, Christian Brauner, Eric Biggers,
	Ard Biesheuvel, Zipeng Zhang, Jeff Layton, Kees Cook
  Cc: Amir Goldstein, ecryptfs, linux-kernel

s/vale/value/

Signed-off-by: Thorsten Blum <thorsten.blum@linux.dev>
---
 fs/ecryptfs/crypto.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/fs/ecryptfs/crypto.c b/fs/ecryptfs/crypto.c
index 3b59346d68c5..585bad6577ed 100644
--- a/fs/ecryptfs/crypto.c
+++ b/fs/ecryptfs/crypto.c
@@ -72,7 +72,7 @@ static int ecryptfs_crypto_api_algify_cipher_name(char **algified_name,
 
 /**
  * ecryptfs_derive_iv
- * @iv: destination for the derived iv vale
+ * @iv: destination for the derived iv value
  * @crypt_stat: Pointer to crypt_stat struct for the current inode
  * @offset: Offset of the extent whose IV we are to derive
  *

^ permalink raw reply related	[flat|nested] 3+ messages in thread

* [PATCH 2/2] ecryptfs: Drop TODO comment in ecryptfs_derive_iv
  2026-03-30 10:35 [PATCH 1/2] ecryptfs: Fix typo in ecryptfs_derive_iv function comment Thorsten Blum
@ 2026-03-30 10:35 ` Thorsten Blum
  2026-04-02 22:32 ` [PATCH 1/2] ecryptfs: Fix typo in ecryptfs_derive_iv function comment Tyler Hicks
  1 sibling, 0 replies; 3+ messages in thread
From: Thorsten Blum @ 2026-03-30 10:35 UTC (permalink / raw)
  To: Tyler Hicks, Thorsten Blum, Christian Brauner, Eric Biggers,
	Ard Biesheuvel, Kees Cook, Amir Goldstein, Jeff Layton
  Cc: Damien Le Moal, Zipeng Zhang, ecryptfs, linux-kernel

Remove the TODO from 2006.

Signed-off-by: Thorsten Blum <thorsten.blum@linux.dev>
---
 fs/ecryptfs/crypto.c | 4 ----
 1 file changed, 4 deletions(-)

diff --git a/fs/ecryptfs/crypto.c b/fs/ecryptfs/crypto.c
index 585bad6577ed..92fbd28e9178 100644
--- a/fs/ecryptfs/crypto.c
+++ b/fs/ecryptfs/crypto.c
@@ -89,10 +89,6 @@ void ecryptfs_derive_iv(char *iv, struct ecryptfs_crypt_stat *crypt_stat,
 		ecryptfs_printk(KERN_DEBUG, "root iv:\n");
 		ecryptfs_dump_hex(crypt_stat->root_iv, crypt_stat->iv_bytes);
 	}
-	/* TODO: It is probably secure to just cast the least
-	 * significant bits of the root IV into an unsigned long and
-	 * add the offset to that rather than go through all this
-	 * hashing business. -Halcrow */
 	memcpy(src, crypt_stat->root_iv, crypt_stat->iv_bytes);
 	memset((src + crypt_stat->iv_bytes), 0, 16);
 	snprintf((src + crypt_stat->iv_bytes), 16, "%lld", offset);

^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH 1/2] ecryptfs: Fix typo in ecryptfs_derive_iv function comment
  2026-03-30 10:35 [PATCH 1/2] ecryptfs: Fix typo in ecryptfs_derive_iv function comment Thorsten Blum
  2026-03-30 10:35 ` [PATCH 2/2] ecryptfs: Drop TODO comment in ecryptfs_derive_iv Thorsten Blum
@ 2026-04-02 22:32 ` Tyler Hicks
  1 sibling, 0 replies; 3+ messages in thread
From: Tyler Hicks @ 2026-04-02 22:32 UTC (permalink / raw)
  To: Thorsten Blum
  Cc: Christian Brauner, Eric Biggers, Ard Biesheuvel, Zipeng Zhang,
	Jeff Layton, Kees Cook, Amir Goldstein, ecryptfs, linux-kernel

On Mon, 30 Mar 2026 12:35:15 +0200, Thorsten Blum wrote:
> s/vale/value/

Thank you! Both patches have 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/2] ecryptfs: Fix typo in ecryptfs_derive_iv function comment
      https://git.kernel.org/tyhicks/ecryptfs/c/c8717a7fa5d2ea9048f398c16ab61d8e4c8f83b8
[2/2] ecryptfs: Drop TODO comment in ecryptfs_derive_iv
      https://git.kernel.org/tyhicks/ecryptfs/c/cd3b3094df0ee0f147957e7a7a1103990fdd6641

Tyler

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2026-04-02 22:32 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-03-30 10:35 [PATCH 1/2] ecryptfs: Fix typo in ecryptfs_derive_iv function comment Thorsten Blum
2026-03-30 10:35 ` [PATCH 2/2] ecryptfs: Drop TODO comment in ecryptfs_derive_iv Thorsten Blum
2026-04-02 22:32 ` [PATCH 1/2] ecryptfs: Fix typo in ecryptfs_derive_iv function comment Tyler Hicks

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox