On Mon, May 01, 2017 at 05:15:24PM -0700, Eric Biggers wrote: > From: Eric Biggers <ebiggers@xxxxxxxxxx> > > [RFC for now since the kernel fixes are currently sitting in fscrypt/master and > haven't been merged to Linus's tree yet; I'll resend this once they're merged] > > Add a test which creates many similarly-named files in an encrypted > directory, then verifies they can be deleted without access to the > encryption key. This is a regression test for two related bugs which > caused presented names to "collide" and point to the wrong inodes. > > Cc: linux-fscrypt@xxxxxxxxxxxxxxx > Signed-off-by: Eric Biggers <ebiggers@xxxxxxxxxx> The test looks fine to me, it fails on ext4 with rm reporting "Structure needs cleaning". One minor issue is that rm reports too many failures (around 98000 lines in my test) +rm: cannot remove '/mnt/testarea/scratch/edir/_SET,lbqVCybotHoHkKrq4xM7FtbuBCLY': Structure needs cleaning and that makes the output diff a bit harder to review, and takes too much space in 500.out.bad file unnecessarily (11M). Better to trim or limit the error log somehow, e.g. using "head -n 10"? Thanks, Eryu -- To unsubscribe from this list: send the line "unsubscribe linux-fscrypt" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
|