On Sat, Mar 26, 2016 at 01:10:07AM -0500, Eric Biggers wrote:
> Why was the XTS tweak initialization changed in commit 0b81d0779072 ("fs
> crypto:
> move per-file encryption from f2fs tree to fs/crypto")?
>
> Old code:
>
> memcpy(xts_tweak, &index, sizeof(index));
> memset(&xts_tweak[sizeof(index)], 0,
> F2FS_XTS_TWEAK_SIZE - sizeof(index));
>
> New code:
> memcpy(xts_tweak, &inode->i_ino, sizeof(index));
> memset(&xts_tweak[sizeof(index)], 0,
> FS_XTS_TWEAK_SIZE - sizeof(index));
>
> Now the XTS tweak is the same for all pages of each inode.
Thank you for catching this.
I've checked several times, but turns out I missed something tho. :(
Let me write a patch to fix this.
Thanks,
------------------------------------------------------------------------------
Transform Data into Opportunity.
Accelerate data analysis in your applications with
Intel Data Analytics Acceleration Library.
Click to learn more.
http://pubads.g.doubleclick.net/gampad/clk?id=278785351&iu=/4140
_______________________________________________
Linux-f2fs-devel mailing list
Linux-f2fs-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linux-f2fs-devel
